Re: Fwd: [PHP-DEV] RC1
| From: | Timm Friebe | Date: | Thu, 26 Feb 2004 00:33:10 +0000 |
| Subject: | Re: Fwd: [PHP-DEV] RC1 | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-8134@lists.php.net to get a copy of this message | ||
On Wed, 2004-02-25 at 15:41, Andi Gutmans wrote:
> Quick reminder...
> If you have any bugs to fix, please fix them in the coming days.
The following:
------------------------------------------------------------------
<?php
class Base {
public $id= 0;
}
class Child extends Base {
public $prop= '';
}
$p= &new Reflection_Property('Child', 'id');
echo 'Child::$id => ', $p->getDeclaringClass()->getName(), "\n";
$p= &new Reflection_Property('Child', 'prop');
echo 'Child::$prop => ', $p->getDeclaringClass()->getName(), "\n";
?>
------------------------------------------------------------------
currently prints out:
Child::$id => Base
Child::$prop => Base
Expected output is:
Child::$id => Base
Child::$prop => Child
Fix is attached. It also includes two more fixes. The first one replaces
searching for a zval** in ce->properties_info by passing a char* (we
will actually find something then:)). The second one fixes wrong
reporting of getDeclaringClass() in a situation in which the above test
script would have a private $prop in the Base class.
- Timm
Attachment: [text/x-patch] zend_reflection_api.diff
Attachment: [text/x-patch] zend_reflection_api.diff