|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2006-02-09 12:12 UTC] denis at edistar dot com
Description:
------------
I have a base abstract class that defines a property to be
protected and that also checks this runtime using the
reflection api.
My purpose is to enforce the extending class to declare
that property as protected too.
The reflection api returns me wrong information. It
returns always that the property is protected (as defined
in the base class), ignoring the definition of the
extending class.
Reproduce code:
---------------
abstract class enum {
protected $_values;
public function __construct() {
$property = new ReflectionProperty(get_class($this),'_values');
if(!$property->isProtected()) {
throw new Exception("Property _values must be declared as protected");
}
}
}
final class myEnum extends enum {
public $_values = array(
0 => 'No value',
1 => 'Value n.1',
2 => 'Value n.2',
4 => 'Value n.4'
);
}
$x = new myEnum();
Expected result:
----------------
Fatal error: Uncaught exception 'Exception' with message
'Property _values must be declared as protected'
Actual result:
--------------
No exception is thrown
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||
Copyright © 2001-2024 The PHP GroupAll rights reserved. |
Last updated: Thu Nov 21 09:01:32 2024 UTC |
The ReflectionProperty class is correctly checking for myEnum->_values property (not for enum->_values) but it returns the visibilty of enum->_values instead of myEnum->_values. get_class($this) in the enum constructor returns correctly myEnum, so the line "$property = new ReflectionProperty(get_class($this),'_values');" is equivalent to writing "$property = new ReflectionProperty('myEnum','_values');". The problem remains also if i write this code outside the classes definitions: $property = new ReflectionProperty('myEnum','_values'); if($property->isProtected()) { throw new Exception("Property _values must be declared as protected"); } Thank you, Denis