Bug #55475 [Csd->Asn]: is_a() triggers autoloader

From: Date: Thu, 15 Sep 2011 11:00:29 +0000
Subject: Bug #55475 [Csd->Asn]: is_a() triggers autoloader
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-163556@lists.php.net to get a copy of this message 
Edit report at https://bugs.php.net/bug.php?id=55475&edit=1

 ID:                 55475
 Updated by:         dmitry@php.net
 Reported by:        mads at gartneriet dot dk
 Summary:            is_a() triggers autoloader
-Status:             Closed
+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:

Reverted before the common decision.


Previous Comments:
------------------------------------------------------------------------
[2011-09-15 10:59:23] dmitry@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] dmitry@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] dmitry@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

------------------------------------------------------------------------
[2011-08-29 07:15:47] tyrael@php.net

"note the "FALSE otherwise" ..."

note "if the object" ...

Tyrael

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


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

Thread (51 messages)

« previous php.bugs (#163556) next »