|
|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2007-03-15 19:35 UTC] phpbugs at thequod dot de
Description:
------------
If a member of an object is not defined and "gets initialized" by
PHP after/during the overloading process, a notice ("Indirect
modification of overloaded property") gets triggered when PHP has to
initialize it as an array type.
It makes no difference, if __get() returns by reference instead (a
ref to a null value), as Tony stated in
http://bugs.php.net/bug.php?id=40823 (the code in his comment does
not work (anymore?)).
Background: we have a base class "Plugin" in our project and it uses
__get() for some properties and returns null otherwise.
Now, if some user-created plugin does
$this->foo[] = 'bar'
it will create this notice - which is quite confusing (if the
derived plugin does not "init" $foo with "var $foo").
Can't PHP internally do "$this->test = array()"?
(This may likely be a dupe of http://bugs.php.net/bug.php?id=39337 -
but that one got quite confusing, so this one here may become
a "reference bogus bug" for this issue at least.)
Reproduce code:
---------------
<?php
class A
{
function __get($name)
{
echo "__get() called.\n";
}
function A()
{
$this->test[] = 'foo';
}
}
$A = new A();
?>
Expected result:
----------------
__get() called.
Actual result:
--------------
__get() called.
Notice: Indirect modification of overloaded property A::$test has no
effect in foo.php on line 12
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Sat Nov 29 12:00:01 2025 UTC |
>>Can't PHP internally do "$this->test = array()"? >No, it can't do your job for you. You have to do it in the __get() method. 1. It also won't work with function &__get($name) { echo "__get() called.\n"; $r = array(); return $r; } 2. It would not make sense to init any unknown/unhandled property as "array()" anyway 3. Your comment works, if you remove the "(array)" typecast in &__get() - it will cause "Only variable references should be returned by reference" otherwise 4. I'm not looking for support, but state that "$this->foo[]" is not supported in overloading - in contrast what you would expect Please note that this is different from bug 40823, where _any_ property (which goes through __get()) gets handled as Foo::array! What I want to work is: Only handle some members as overloaded properties (e.g. $foo and $bar), but let all others work as intended - and that means "$A->foobar[]=" should work, just like "$foobar[]=" does. Please make sure you really understand what the problem IMHO is here! At least this should get closed as "Wont fix" and documented on http://php.net/manual/en/language.oop5.overloading.php. See also http://php.net/manual/en/language.oop5.overloading.php#73512