#23619 [Bgs->Opn]: set_error_handler registered hdler not called for object instances

ID: 23619 User updated by: ldemailly at qualys dot com -Summary: set_error_handler registered hdler not (always) called Reported By: ldemailly at qualys dot com -Status: Bogus +Status: Open Bug Type: Scripting Engine problem Operating System: Linux 2.4.20 PHP Version: 4CVS-2003-05-13 (stable) New Comment: Aha, I managed to reduce the repro script dramatically : <?php ini_set("error_reporting" , E_ALL); class TestErrorHdl { // Will not be called !! function errorHandler($errno, $errstr, $errfile, $errline) { echo("ERROR HANDLER CALLED (EXPECTED)\n"); } function runTest() { $res = set_error_handler(array(&$this, "errorHandler")); echo("set_error_handler done retcode=" . $res . "\n"); $bar = $foo; } } $t = new TestErrorHdl(); $t->runTest(); ?> --- output (buggy, with current 4.3 php cli versions): ---- $ php ErrorBug.php set_error_handler done retcode=1 Notice: Undefined variable: foo in /home/dl/test_err_tiny/ErrorBug.php on line 12 --- output (expected) : ---- set_error_handler done retcode=1 ERROR HANDLER CALLED (EXPECTED) [note that with a longer script it does *most of the time* work as expected, so it's not like the feature of having an instance method for a error handler is not supported at all...] Previous Comments: ------------------------------------------------------------------------ [2003-05-14 10:42:42] sniper@php.net You're propably doing something wrong. As long as you can't provide a _SHORT_ example script which clearly shows the problem, we'll assume that. ------------------------------------------------------------------------ [2003-05-13 21:26:28] ldemailly at qualys dot com set_error_handler registered not called: very nasty/triky bug to nail/reproduce: I'm now at the point where if I try to minimize further the test scripts, the bug goes away: for instance, removing the "require q.inc" where q.inc is an empty file, or changing code that is *after* the offending code makes the bug go away (hides it), so I suspect some nasty problem in the engine, because it seems memory usage and/or complexity of the script to be parsed affects the outcome. to repro: copy paste, uudecode; gtar xvfz test_err.tgz cd test_err; php ErrorBug.php it will output "INSIDE test_should_error_out()" but not "ERROR HANDLER CALLED" as it should if you look at PHPUnit/TestCase.php line 158 and 147 the error is raised (as the Notice: Undefined variable: foo in /home/dl/dev/cvs/qualys/web/dev/current/internal/test_err/ErrorBug.php on line 7 demonstrates) but the handler is not called... if you edit BugTest1.php to remove the require 'q.inc' for instance, you will now see "ERROR HANDLER CALLED" even though q.inc is empty and should thus make no difference at all ! ps: the code has been modified from pear's phpunit module with a submitted patch for error handling; which works 'sometimes' (most of the time actually). in the tgz below, I removed a lot of stuff to make the files as small as possible... (incl headers/copyright notices, etc...)) I spent many many hours trying to narrow it down further (having a simpler script) but didn't manage, sorry for having a large set of files needed ... This is with the latest cvs (2003-05-13) on redhat linux 9 next message will be the .uu, if you're having any trouble, please email and I'll (re)send the repro scripts ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=23619&edit=1