Bug #71240 [Com]: array_filter byref value in callable no longer working
[email protected] ("donatj at gmail dot com")
| Newsgroups | php.bugs |
|---|---|
| Message-ID | <[email protected]> |
Edit report at https://bugs.php.net/bug.php?id=71240&edit=1
ID: 71240
Comment by: donatj at gmail dot com
Reported by: donatj at gmail dot com
Summary: array_filter byref value in callable no longer
working
Status: Not a bug
Type: Bug
Package: *General Issues
PHP Version: 7.0.1
Block user comment: N
Private report: N
New Comment:
Also the use case for this - I'm array_filter-ing already, not with a hardcoded true but actual logic, and as the code is already looping over items having a second loop through my array with array_map is silly.
Previous Comments:
------------------------------------------------------------------------
[2015-12-29 22:39:22] donatj at gmail dot com
I still argue it's a bug. It's a behaviour that goes way back to at least 4.3.0 https://3v4l.org/iTqYI
If nothing else, the change should be noted in the array_filter documentation, but I'd still argue it should be fixed.
------------------------------------------------------------------------
[2015-12-29 22:22:25] [email protected]
@requinix: Actually I'm just stupid, you are totally right. I got confused with which output is which.
------------------------------------------------------------------------
[2015-12-29 21:03:42] [email protected]
Fair enough. It just looked like one more of the various inconsistencies with references that was fixed with 7.
------------------------------------------------------------------------
[2015-12-29 20:58:52] [email protected]
@requinix: The issue here is that is does the by-reference modification, while it should not. A similar, but slightly different, bug exists in HHVM as well: https://3v4l.org/dp6Lt
------------------------------------------------------------------------
[2015-12-29 20:57:12] [email protected]
I don't see anything to suggest that was supported in the first place. There aren't even user comments doing that.
Besides, if you want to alter the array values then array_map() is more appropriate.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=71240
--
Edit this bug report at https://bugs.php.net/bug.php?id=71240&edit=1