Doc #77100 [Com]: Inconsistent behavior between constants and magic constants

Edit report at https://bugs.php.net/bug.php?id=77100&edit=1 ID: 77100 Comment by: a at b dot c dot de Reported by: fabien dot villepinte at gmail dot com Summary: Inconsistent behavior between constants and magic constants Status: Assigned Type: Documentation Problem Package: Scripting Engine problem PHP Version: Irrelevant Assigned To: cmb Block user comment: N Private report: N New Comment: That is consistent with variable names: just as you can create a variable that has a name impossible to use as a literal (e.g., ${"$"}) so too you can create a constant that can't be referred to literally. Previous Comments: ------------------------------------------------------------------------ [2018-11-04 14:58:11] cmb@php.net @nikic Thanks! Changing to doc problem. ------------------------------------------------------------------------ [2018-11-04 12:44:05] nikic@php.net I'm inclined to mark this won't fix. Magic constants are just keywords that have syntactical similarity to constants. I don't think we have any business checking that constant names do not match keywords. If we forbid define('__DIR__'), should we also forbid define('function')? Ultimately everything here works just fine in the sense that you can define that constant and also look it up using constant(). You can't access it with constant syntax because the __DIR__ keyword takes precedence, but there's any number of other reasons why such an access may not be possible. For example, you can also do define('123', 'foo') and then access it using constant('123'), but for obvious reasons it will not change the behavior of directly writing 123 in code. ------------------------------------------------------------------------ [2018-11-04 11:21:23] cmb@php.net > I'd suggest prepopulating the constants table with them only I > don't know how something like constant('__LINE__') could work. Prepopulating the constants table would require to update it frequently during run time to be consistent with the compile time constants. Besides being inefficient, it would also change “constant” values. It might be preferable to not allow any of the predefined magic constants (or maybe all constant names with two leading and trailing underscores) to be used with define(), defined() and constant(). Either way would break BC, though. ------------------------------------------------------------------------ [2018-11-03 21:27:48] a at b dot c dot de To continue that: Which means that when run time comes around the names are free to be redefined as user constants. But the literal identifiers aren't in the code any more so the only way they can be accessed is through the constant('') construct. The compile-time evaluation of magic constants also means define('__DIR__', 0); is legal, but const __DIR__ = 0; is not, since the latter definition happens at compile time too. Maybe a check (c|sh)ould be made that users aren't trying to define() magic constants themselves? I'd suggest prepopulating the constants table with them only I don't know how something like constant('__LINE__') could work. ------------------------------------------------------------------------ [2018-11-03 16:54:34] cmb@php.net FWIW, the docs[1] say: | All these "magical" constants are resolved at compile time, | unlike regular constants, which are resolved at runtime. [1] <http://php.net/manual/en/language.constants.predefined.php> ------------------------------------------------------------------------ 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 https://bugs.php.net/bug.php?id=77100 -- Edit this bug report at https://bugs.php.net/bug.php?id=77100&edit=1