Edit report at http://bugs.php.net/bug.php?id=52969&edit=1
ID: 52969 Comment by: jesse_7254 at yahoo dot com Reported by: ircmaxell at yahoo dot com Summary: This defined in static call from within another class Status: Bogus Type: Bug Package: Class/Object related Operating System: linux PHP Version: 5.3.3 Block user comment: N New Comment: A bogus bug is a bogus bug, no matter the opinion of StackOverflow amateurs. Previous Comments: ------------------------------------------------------------------------ [2010-10-03 18:28:51] + at ni-po dot com @ircmaxell: No, you don't need to modify all your code and add that line. Or at least you don't need to do so if you don't already have this line in every method: if (!isset($this)) throw new LogicException('Called non-static method statically'); In PHP you can't rely on $this being set. But still you do, because an unset $this is a real edge case. But I still do think that this feature *is* a bug and shall be removed. It is absolutely illogical that $this isn't instanceof self. ------------------------------------------------------------------------ [2010-10-03 00:37:43] gnuffo1 at gmail dot com Does it really mean you must put that line in every method? It's not like you ever accidentally had a problem with it before is it? ------------------------------------------------------------------------ [2010-10-02 21:05:25] ircmaxell at yahoo dot com Close it if you wish, but I do honestly believe this is a major bug... This means that you cannot trust that $this is an instance of the class it's used in. So does that mean that we must add: ($this instanceof self) or throw new Exception('called from another class'); to all of our methods? Remember, any sufficiently advanced bug is indistinguishable from a feature (no matter if it is documented or not). But don't be afraid to call it what it really is... Just because it's documented, doesn't mean it isn't a bug... Again, IMHO... ------------------------------------------------------------------------ [2010-10-02 20:07:43] cataphr...@php.net Closing as bogus. As you have pointed out, this is documented. ------------------------------------------------------------------------ [2010-10-02 15:51:04] gnuffo1 at gmail dot com It has been pointed out to me that this is actually documented: http://us2.php.net/manual/en/language.oop5.basic.php Under example 2 and specifically the lines: $b = new B(); $b->bar(); ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/bug.php?id=52969 -- Edit this bug report at http://bugs.php.net/bug.php?id=52969&edit=1