Bug->Doc #55038 [Opn]: inconsistent error-handling for $this with closure

From: Date: Tue, 15 Jul 2014 11:22:01 +0000
Subject: Bug->Doc #55038 [Opn]: inconsistent error-handling for $this with closure
References: 1  Groups: php.doc.bugs 
Request: Send a blank email to doc-bugs+get-11372@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=55038&edit=1 ID: 55038 Updated by: yohgaki@php.net Reported by: rasmus at mindplay dot dk -Summary: inconsistent error-handling for $this +Summary: inconsistent error-handling for $this with closure Status: Open -Type: Bug +Type: Documentation Problem Package: Class/Object related Operating System: Win7 PHP Version: 5.3.6 Block user comment: N Private report: N New Comment: Closure is implemented by class. Therefore, this is limitation. If this is not documented, it should be written somewhere. Previous Comments: ------------------------------------------------------------------------ [2012-09-06 10:27:28] dexen dot devries at gmali dot com Note, the closure and creation of another object instance are irrelevant. Two simpler reproduction scripts -- one uses anonymous function, the other plain function. <?php $foo = function($this) { var_dump($this); // this will work! var_dump($this->bar); // this will blow up }; $foo(new stdclass); ?> <?php function bar($this) { var_dump($this); // this will work! var_dump($this->bar); // this will blow up } bar(new stdclass); ?> Expected result: both uses of $this and $this->bar behave in similar way. Actual result: PHP complains only on $this->bar. ------------------------------------------------------------------------ [2011-06-12 03:53:57] rasmus at mindplay dot dk Description: ------------ $this is not generally allowed outside an object context, but sometimes it is. I discovered this while trying to write a cheeky little base-class that would allow you to decorate objects with new methods, at run-time. I would have gotten away with it, too - and I still could of course, only I would have to break from the convention of $this, and naming the context-object something else, which kind of sucks. Test script: --------------- <?php $foo = function($this) { var_dump($this); // this will work! }; $foo('bar'); // now an object: class Test { public $foo = 'bar'; } $test = new Test; // and another closure: $ouch = function($this) { var_dump($this->bar); // this will blow up }; $ouch($test); Expected result: ---------------- The two examples should either fail consistently, or succeed consistently. From my point of view, why should I not be allowed to have a local variable named $this if I wanted to? There's nothing special or magical about this variable, besides the fact that it gets automatically assigned on call. Actual result: -------------- The second example fails. ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=55038&edit=1

« previous php.doc.bugs (#11372) next »