ID: 49105 Updated by: j...@php.net Reported By: atrauzzi at gmail dot com Status: Bogus Bug Type: Class/Object related Operating System: Linux PHP Version: 5.3.0 New Comment:
If you disagree, feel free to bring the issue up on: intern...@lists.php.net And discuss it there, this bug system is not a discussion forum. Previous Comments: ------------------------------------------------------------------------ [2009-07-30 13:25:20] atrauzzi at gmail dot com As an addition, the example provided by "mail at dropdev dot org" contains a bug. He uses static:: yet ends up with a reference to MasterAlarm. If he wanted the calls to consistently reflect MasterAlarm, he should be using self::, as the moment any subclass declares $state, the method showMasterAlarmState() breaks by way of the subclass property. That demonstrates how the current behaviour contradicts the expectations made of late static binding and why I suggest at the very least an error should be thrown noting an undefined property. This in itself is classic proof of how this ambiguity is resulting in people not only accepting incorrect design convention, but also advocating it because they have come to depend on it. The whole idea of forcing a declaration of a new property in a subclass is undesirable. I would encourage anyone reading to revisit the principles behind why somebody would want to inherit from a base class and the advantages of using self:: over static:: explicitly rather than relying on implicit behaviour that causes poor design. At the very least, this issue deserves more than a stonewall tactic and could likely use a formal and thoughtful review.) ------------------------------------------------------------------------ [2009-07-30 12:26:47] atrauzzi at gmail dot com I half expected this to happen as this is how the vast majority of issues here are handled. It would be nice if there was a process to outline *why* this behaviour shouldn't be corrected. A more advanced review is definitely warranted here. That is - without simply cut-and-pasting a snippet telling people to look at the documentation. Which is obviously a stonewall tactic to reduce bug workload. In the strictest sense, any bug will not be documented and the current behaviour is no exception to that. I would be pleasantly surprised to see this actually mentioned in the documentation. Consider that if one wanted to use the parent class property, they simply would have to use self:: in the parent or parent:: in the child. Which makes it an *explicit* design, not implied limitation that has to be worked around. At present, this definite bug requires the use of what could only be described as a workaround and I do not feel it has been properly considered. Being able to post source code that makes use of the bug doesn't validate it. Because of this it is an angle of functionality not considered or explicitly handled. Instead, a design implied by accident is accepted, in spite of the fact that it is of limited use. I'm surprised as I'm sure issues requesting late static binding were flagged "bogus" in the past. How long will it be before this issue is re-reported again?) ------------------------------------------------------------------------ [2009-07-30 10:19:38] j...@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 ------------------------------------------------------------------------ [2009-07-30 04:26:24] mail at dropdev dot org You're confusing two similar, yet different concepts, Inheritance and Extension. Also, PHP is doing it perfect, since you actually inherit a static variable (that all point to the same memory point.) think of this example: <?php abstract class MasterAlarm { protected static $state = ""; public static function alert() { static::$state = "ALERT"; } public static function safe() { static::$state = "safe"; } public static function showMasterAlarmState() { return(static::$state); } } ?> and then extend it: <?php class Bell extends MasterAlarm { } class Button extends MasterAlarm { } ?> and the executing code <?php Light::safe(); Light::showMasterAlarmState(); // "safe" Button::alert(); Button::showMasterAlarmState(); // "ALERT" Light::showMasterAlarmState(); // "ALERT" Button::safe(); Light::showMasterAlarmState(); // "safe" ?> ------------------------------------------------------------------------ [2009-07-30 00:42:01] atrauzzi at gmail dot com Description: ------------ Subclass properties should be duplicated from the parent class, not passed through to the parent class. The current behaviour is redundant and possibly misleading. Especially with the great selection of scopes in PHP (parent::, static::, self::). The current workaround is to dump everything into an array on the parent, keyed by subclass. This knocks out cohesive design and restricts the potential for some fairly neat designs. At the very least - to prevent accidental misuse - a subclass should be throwing an error when trying to access a property that doesn't exist, but does exist on a parent. Really what should be happening though, is a child should be getting its own copy of each property defined in the parent. Reproduce code: --------------- abstract class Parent { protected static $message = "UNTOUCHED"; public static function yeah() { static::$message = "YEAH"; } public static function nope() { static::$message = "NOPE"; } public static function lateStaticDebug() { return(static::$message); } } class Child extends Parent { } Expected result: ---------------- Parent::yeah(); Parent::lateStaticDebug(); // Return "YEAH" Child::nope(); Child::lateStaticDebug(); // Return "NOPE" Parent::yeah(); Child::lateStaticDebug() // Return "NOPE" Actual result: -------------- Parent::yeah(); Parent::lateStaticDebug(); // Returns "YEAH" Child::nope(); Child::lateStaticDebug(); // Returns "NOPE" Parent::yeah(); Child::lateStaticDebug() // Returns "YEAH" ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=49105&edit=1
