#29716 [WFx]: __call with wrong no. of arguments prevents simple BC wrappers.
| From: | alan_k@php.net | Date: | Fri, 20 Aug 2004 03:51:51 +0000 |
| Subject: | #29716 [WFx]: __call with wrong no. of arguments prevents simple BC wrappers. | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-64989@lists.php.net to get a copy of this message | ||
ID: 29716
Updated by: alan_k@php.net
-Summary: alan_k
Reported By: alan at akbkhome dot com
Status: Wont fix
Bug Type: Zend Engine 2 problem
Operating System: *
PHP Version: 5.*
New Comment:
changing title (oops)
Previous Comments:
------------------------------------------------------------------------
[2004-08-20 05:50:22] alan_k@php.net
changing status to reflect situation.
Probably the best workaround to offer overload PHP4/PHP5 compatiblity
is to use eval to create a base class eg.
eval('
class PHP4_Overload {
function __call($method,$args,&$return) {
return $this->_call($method,$args,$return);;
}
}
');
------------------------------------------------------------------------
[2004-08-17 09:25:48] helly@php.net
Btw. ext/overload:
This extension is EXPERIMENTAL. The behaviour of this extension --
including the names of its functions and anything else documented about
this extension -- may change without notice in a future release of PHP.
Use this extension at your own risk
------------------------------------------------------------------------
[2004-08-17 09:18:32] helly@php.net
Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
http://www.php.net/manual/ and the instructions on how to
report
a bug at http://bugs.php.net/how-to-report.php
We no longer support ext/overload (and ext/aggregation). In PHP 5
__autoload was reimplemented as a new different feature even though it
is in some manner meant to be able to replace the old behavior. But it
is meant to do it etter and offer more. Hence unfortunatley those two
are different things and yes we broke BC.
------------------------------------------------------------------------
[2004-08-17 08:15:28] derick@php.net
Marcus breaks it, marcus should fix it :)
Also, can you please add a test case for this?
thanks
------------------------------------------------------------------------
[2004-08-17 06:21:19] alan_k@php.net
Relivant change
http://cvs.php.net/diff.php/ZendEngine2/zend_compile.c?ws=0&r1=1.567&r2=1.567.2.1&ty=u
I suggest testing for > 1, rather than != 2 on call, as that's the only
one who's API changed in PHP4->5
------------------------------------------------------------------------
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/29716
--
Edit this bug report at http://bugs.php.net/?id=29716&edit=1