#42115 [Opn->Csd]: Should not allow static keyword before visibility keyword

ID: 42115 Updated by: rquadling@php.net Reported By: mattsch at gmail dot com -Status: Open +Status: Closed Bug Type: Documentation problem Operating System: Gentoo Linux PHP Version: 5.2.3 New Comment: This bug has been fixed in the documentation's XML sources. Since the online and downloadable versions of the documentation need some time to get updated, we would like to ask you to be a bit patient. Thank you for the report, and for helping us make our documentation better. Previous Comments: ------------------------------------------------------------------------ [2007-07-27 08:57:28] rquadling@php.net Code: http://pastie.caboo.se/82738 Output: http://pastie.caboo.se/82739 It seems that the static can really go either side of the visibility keyword for all visibilities. I can confirm that this is the case since V5.0.0 (just downloaded that from (3 years old this month). ------------------------------------------------------------------------ [2007-07-27 07:12:46] derick@php.net Documentation problem it is ------------------------------------------------------------------------ [2007-07-27 03:33:29] mattsch at gmail dot com I don't argue that php is a loosely typed language but it does have rules and it should either adhere by them or change the rules. Currently the documentation says that static must come after visibility: http://www.php.net/manual/en/language.oop5.static.php "The static declaration must be after the visibility declaration." So either emit an E_STRICT warning or change the documentation to include this syntax as well. ------------------------------------------------------------------------ [2007-07-27 02:27:14] judas dot iscariote at gmail dot com No, PHP is a dynamic language, I dont see why this should raise any kind of error 1. does not cause any problem. 2.it is perfectly valid code. ------------------------------------------------------------------------ [2007-07-26 20:14:20] mattsch at gmail dot com Description: ------------ Contrary to the documentation, you're allowed to put the static keyword before the visibility keyword. Shouldn't this throw an E_STRICT warning? Reproduce code: --------------- <? class foo { static public function bar($foo){ echo $foo; } } foo::bar('test'); ?> Expected result: ---------------- E_STRICT: You cannot declare static before a visibility keyword. Actual result: -------------- Echoes "test" without warning. ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=42115&edit=1