Bug #50688 [Asn]: Using exceptions inside usort() callback function causes a warning

From: Date: Sat, 18 Jul 2015 14:17:54 +0000
Subject: Bug #50688 [Asn]: Using exceptions inside usort() callback function causes a warning
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-194532@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=50688&edit=1 ID: 50688 Updated by: nikic@php.net Reported by: jcampbell at remindermedia dot com Summary: Using exceptions inside usort() callback function causes a warning Status: Assigned Type: Bug Package: Arrays related Operating System: Fedora Core 12 PHP Version: 5.*, 6 Assigned To: stas Block user comment: N Private report: N New Comment: This has been fixed in PHP 7. Previous Comments: ------------------------------------------------------------------------ [2013-08-19 19:04:24] m at rtin dot so Sorry the use () isn't relevant, I forgot to remove it when simplifying my test case ------------------------------------------------------------------------ [2013-08-19 18:23:17] m at rtin dot so I ran into a similar issue, i'm sure it'll require the same patch as it's the backtrace causing the problem but worth noting it doesn't require an exception to trigger this, just a backtrace. $ cat usort.php <?php set_error_handler(function($errno, $errstr) { $bt = debug_backtrace(); var_dump($errstr); }); $arr = [1, 2]; usort($arr, function($a, $b) use ($arr) { trigger_error('test'); return $a > $b; }); $ php usort.php string(4) "test" string(59) "usort(): Array was modified by the user comparison function" ------------------------------------------------------------------------ [2013-07-09 07:21:19] jakub dot lopuszanski at nasza-klasa dot pl I'd like to add, that you do not have to throw an exception to get this warning. Mere creating it, also triggers the warning, as in: <?php function comp($a,$b){ @new Exception("dupa"); } $a =array(1,2,3); usort($a, 'comp'); var_dump($a); ?> PHP Warning: usort(): Array was modified by the user comparison function in /home/jlopuszanski/test.php on line 6 ------------------------------------------------------------------------ [2013-06-17 11:11:07] andrejs dot verza at gmail dot com Php 5.4.16 also fails with this. Still the same status for 3 and a half years old bug?! ------------------------------------------------------------------------ [2012-08-08 17:53:58] mbrowne83 at gmail dot com This will probably be obvious to most, but I just wanted to mention that you can always prefix the usort function with the @ symbol to prevent the warning...of course that would also suppress any other types of notices or warnings that might occcur anywhere within the sorting function... ------------------------------------------------------------------------ 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=50688 -- Edit this bug report at https://bugs.php.net/bug.php?id=50688&edit=1

« previous php.bugs (#194532) next »