Req #53770 [NEW]: __set Doesn't Allow Pass By Reference (Even On Objects) And Isn't Called On Ref

From: Date: Mon, 17 Jan 2011 21:50:01 +0000
Subject: Req #53770 [NEW]: __set Doesn't Allow Pass By Reference (Even On Objects) And Isn't Called On Ref
Groups: php.bugs 
Request: Send a blank email to php-bugs+get-157965@lists.php.net to get a copy of this message
From:             
Operating system: All
PHP version:      5.3.5
Package:          Class/Object related
Bug Type:         Feature/Change Request
Bug description:__set Doesn't Allow Pass By Reference (Even On Objects) And Isn't Called
On Ref

Description:
------------
When using __set($strProp, &$strValue), you are given the error: 

Method foo::__set() cannot take arguments by reference

If, you use $obj->property =& $that it will not call __set at all, but
rather auto-create 'property';



Not a major issue, but at the least the __set not being called may need to
be fixed.

Test script:
---------------
	class foo {
		private $var = NULL;
		function __set($strProp, $strValue) {
			var_dump('here');
			unset($strValue);
		}
	}
	
	class bar {
		var $that = 'that';
	}
	
	error_reporting(E_STRICT);
	
	$that = 'that';
	
	$foo = new foo();
	
	$foo->Property =& $that;
	var_dump($foo->Property);
	
        var_dump($that);
	
	$bar = new bar();
	$foo->Property =& $bar;
	
	var_dump($bar);

Expected result:
----------------
string(4) "here" 
undefined variable: $bar

Actual result:
--------------
string(4) "that"
object(bar)#7 (1) { ["that"]=> string(4) "that" } 

-- 
Edit bug report at http://bugs.php.net/bug.php?id=53770&edit=1
-- 
Try a snapshot (PHP 5.2):            http://bugs.php.net/fix.php?id=53770&r=trysnapshot52
Try a snapshot (PHP 5.3):            http://bugs.php.net/fix.php?id=53770&r=trysnapshot53
Try a snapshot (trunk):              http://bugs.php.net/fix.php?id=53770&r=trysnapshottrunk
Fixed in SVN:                        http://bugs.php.net/fix.php?id=53770&r=fixed
Fixed in SVN and need be documented: http://bugs.php.net/fix.php?id=53770&r=needdocs
Fixed in release:                    http://bugs.php.net/fix.php?id=53770&r=alreadyfixed
Need backtrace:                      http://bugs.php.net/fix.php?id=53770&r=needtrace
Need Reproduce Script:               http://bugs.php.net/fix.php?id=53770&r=needscript
Try newer version:                   http://bugs.php.net/fix.php?id=53770&r=oldversion
Not developer issue:                 http://bugs.php.net/fix.php?id=53770&r=support
Expected behavior:                   http://bugs.php.net/fix.php?id=53770&r=notwrong
Not enough info:                     http://bugs.php.net/fix.php?id=53770&r=notenoughinfo
Submitted twice:                     http://bugs.php.net/fix.php?id=53770&r=submittedtwice
register_globals:                    http://bugs.php.net/fix.php?id=53770&r=globals
PHP 4 support discontinued:          http://bugs.php.net/fix.php?id=53770&r=php4
Daylight Savings:                    http://bugs.php.net/fix.php?id=53770&r=dst
IIS Stability:                       http://bugs.php.net/fix.php?id=53770&r=isapi
Install GNU Sed:                     http://bugs.php.net/fix.php?id=53770&r=gnused
Floating point limitations:          http://bugs.php.net/fix.php?id=53770&r=float
No Zend Extensions:                  http://bugs.php.net/fix.php?id=53770&r=nozend
MySQL Configuration Error:           http://bugs.php.net/fix.php?id=53770&r=mysqlcfg



Thread (1 message)

  • stormingphoenix at yahoo dot com
« previous php.bugs (#157965) next »