Edit report at https://bugs.php.net/bug.php?id=55475&edit=1

 ID:                 55475
 Patch added by:     ala...@php.net
 Reported by:        mads at gartneriet dot dk
 Summary:            is_a() triggers autoloader
 Status:             Assigned
 Type:               Bug
 Package:            Scripting Engine problem
 PHP Version:        5.3.7
 Assigned To:        dmitry
 Block user comment: N
 Private report:     N

 New Comment:

The following patch has been added/updated:

Patch Name: is_class_of.txt
Revision:   1316553958
URL:        
https://bugs.php.net/patch-display.php?bug=55475&patch=is_class_of.txt&revision=1316553958


Previous Comments:
------------------------------------------------------------------------
[2011-09-15 11:00:29] dmi...@php.net

Reverted before the common decision.

------------------------------------------------------------------------
[2011-09-15 10:59:23] dmi...@php.net

Automatic comment from SVN on behalf of dmitry
Revision: http://svn.php.net/viewvc/?view=revision&revision=316811
Log: Reverted the fix for #55475 (is_a() triggers autoloader) before the common 
decision

------------------------------------------------------------------------
[2011-09-15 10:00:16] dmi...@php.net

I've committed the revert.is_a.behaviour.to.ignoring.strings.diff by alan at 
akbkhome dot com into 5.3.

5.4 is going to support string argument.

------------------------------------------------------------------------
[2011-09-15 09:58:17] dmi...@php.net

Automatic comment from SVN on behalf of dmitry
Revision: http://svn.php.net/viewvc/?view=revision&revision=316810
Log: Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)

------------------------------------------------------------------------
[2011-09-07 06:30:47] vchernoivan at gmail dot com

I guess it is no use to argue if the behaviuor is correct or not, or how 
precise 
the manual is. Since IT IS BREAKING EXISTING CODE, for me, too.
Before the change
   if (is_a($date,"DateTime"))
       return $date->format(...);
   /// some code handling datetime strings
worked just fine. Now it triggers __autoload and results in completely broken 
page. 
For sure, personally I can  change every piece of MY OWN code. 
But consider users of tons of PHP libraries! 
What do you think, how long will it take to update every piece of them?
Vote for reverting to prior-5.7 behavior until 5.4

------------------------------------------------------------------------


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=55475


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=55475&edit=1

Reply via email to