#28872 [Opn->Bgs]: accessing a __get (getter) method from object value fails
| From: | tony2001@php.net | Date: | Fri, 14 Jan 2005 23:04:03 +0000 |
| Subject: | #28872 [Opn->Bgs]: accessing a __get (getter) method from object value fails | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-72166@lists.php.net to get a copy of this message | ||
ID: 28872
Updated by: tony2001@php.net
Reported By: nick dot rich at textmarketer dot co dot uk
-Status: Open
+Status: Bogus
Bug Type: Class/Object related
Operating System: *
PHP Version: 5.0.0RC3
New Comment:
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
Previous Comments:
------------------------------------------------------------------------
[2004-07-09 00:07:29] csmoak at andrew dot cmu dot edu
This is a bogus bug.
The problem here is that you are trying to access properties in an
improper way--pay attention to order of operations.
The interpreter will operate from left to right--that is, on the line
marked /// fails in the example code, it will first access $test, then
call __get() with the value of (string) $config, which is 'Object'. You
should have also gotten an error similar to:
Notice: Object of class config to string conversion in
C:\php5\test2.php on line 28
which shows that it is doing this cast to a string.
you are expecting it to evaluate something like:
$test->{$config->methodnames[1]};
when, in fact, php evaluates:
{$test->$config}->methodnames[1];
which is not a bug.
------------------------------------------------------------------------
[2004-06-22 09:51:42] nick dot rich at textmarketer dot co dot uk
The point is that the __get ihis being passed an object
ref instead of the value. If you change the test class
to:
Class TestClass
{
private static $data =
array("dog"=>"dog","cat"=>"cat");
function __get($value)
{
return self::$data[$value];
}
}
It will give the same result
------------------------------------------------------------------------
[2004-06-21 20:59:34] helly@php.net
first of all __get() is supposed to return something.
------------------------------------------------------------------------
[2004-06-21 19:35:32] nick dot rich at textmarketer dot co dot uk
Description:
------------
Trying to access a propery (via the get method) using a
value from another object as the method name fails.
See code::
if the line /// fails
is
replaced with
$localVar = $config->methodnames[1];
$test->$localVar
It works correctly
Reproduce code:
---------------
class config
{
public $methodnames = array("dog","cat");
}
class testClass
{
function Cat(){
echo "cat";
}
function Dog(){
echo "dog";
}
function __get($method){
eval("\$this->\$method();");
}
}
class main
{
function __construct(){
$config = new config();
$test = new testClass();
$test->$config->methodnames[1]; /// fails
}
}
$test = new main();
Expected result:
----------------
cat
Actual result:
--------------
Fatal error: Call to undefined method testClass::
Object() in /Users/nicholasrich/Sites/bits/
testclass.php(18) : eval()'d code on line 1
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=28872&edit=1