ID: 45743 User updated by: ilewis at uk dot ibm dot com Reported By: ilewis at uk dot ibm dot com Status: Closed Bug Type: Scripting Engine problem Operating System: Fedora 8 PHP Version: 5.3CVS-2008-08-07 (snap) Assigned To: felipe New Comment:
That seems like a good fix, but the behaviour is now inconsistent with the docs :-) "This function checks if the given property exists in the specified class (and if it is accessible from the current scope)" I guess that needs fixing up as well? Previous Comments: ------------------------------------------------------------------------ [2008-08-18 17:50:44] [EMAIL PROTECTED] This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. I've fixed the property_exists() behavior in 5_3 and HEAD. It now checks the existence, not the accessibility. (like the method_exists()) Thanks. ------------------------------------------------------------------------ [2008-08-18 16:51:29] [EMAIL PROTECTED] In fact, the property_exists() behave differently than method_exists(). ------------------------------------------------------------------------ [2008-08-18 10:20:27] ilewis at uk dot ibm dot com re-opening as I've submitted an updated patch ------------------------------------------------------------------------ [2008-08-15 01:00:01] php-bugs at lists dot php dot net No feedback was provided for this bug for over a week, so it is being suspended automatically. If you are able to provide the information that was originally requested, please do so and change the status of the bug back to "Open". ------------------------------------------------------------------------ [2008-08-07 12:36:22] ilewis at uk dot ibm dot com I've put an updated diff -u output on pastebin: http://pastebin.com/m43078aa5 I've also realised that this fix breaks testcase Zend/tests/bug42211.phpt, but I think this is a bug in the testcase, that is, it is testing for the broken behaviour. I've put a patch for the testcase on pasetbin as well http://pastebin.com/m15c01dec ------------------------------------------------------------------------ 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 http://bugs.php.net/45743 -- Edit this bug report at http://bugs.php.net/?id=45743&edit=1
