php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #25815 __get & __set will not fire if the object contains reference to variable
Submitted: 2003-10-09 21:33 UTC Modified: 2004-05-30 14:46 UTC
From: dan at wep dot net Assigned:
Status: Not a bug Package: Class/Object related
PHP Version: 5 OS: *
Private report: No CVE-ID: None
 [2003-10-09 21:33 UTC] dan at wep dot net
Description:
------------
When a class definition contains any reference to a variable name that will be later used in a __set() (or __get()) operation, the setter (or getter) will not fire.

This variable does not have to be defined or used at all. The code must simply exist.

In the following example code, simply comment out the print($this->myVar); line and you will see the 'setter fired' message.

This is NOT similar to bug #24608 as the element never gets created in this case.

Reproduce code:
---------------
class test {
  function unusedFunction() {
    print($this->myVar);  
  }
  function __set($nm, $vl) {
    print('setter fired');
  }
}

$obj = new test();
$obj->myVar = 'some data';

Expected result:
----------------
'setter fired'


Actual result:
--------------
Nothing.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2003-10-10 12:21 UTC] helly@php.net
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

That\'s correct. We have three types of object bound properties:
1) declared with or without default value
2) implicit declared by usage in source (for BC reasons)
3) virtual through __get() and __set()
 [2004-05-30 14:46 UTC] stas@php.net
I have checked the code with current CVS PHP5, and it displays "setter fired".
 
PHP Copyright © 2001-2020 The PHP Group
All rights reserved.
Last updated: Mon Sep 28 10:01:24 2020 UTC