Bug #66649 [Com]: token_get_all() does not tag new bracket array syntax as T_ARRAY

From: Date: Wed, 05 Feb 2014 20:58:56 +0000
Subject: Bug #66649 [Com]: token_get_all() does not tag new bracket array syntax as T_ARRAY
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-184175@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=66649&edit=1 ID: 66649 Comment by: nikic@php.net Reported by: bishop@php.net Summary: token_get_all() does not tag new bracket array syntax as T_ARRAY Status: Not a bug Type: Bug Package: Scripting Engine problem Operating System: n/a PHP Version: 5.4.24 Block user comment: N Private report: N New Comment: token_get_all exposes PHP's tokenizer/lexer, which is a component those only job is to convert a character stream into a token stream. It does little more than slap a label on certain character sequences. It does not have the ability to distinguish the '[' of an array declaration and the '[' of an array access. If you need this kind of semantic information you should use a PHP parser. Previous Comments: ------------------------------------------------------------------------ [2014-02-05 20:51:27] bishop@php.net Uhm, that's my point. I expect T_ARRAY no longer means "array" literal, but the semantic concept of "array syntax". The documentation even references T_ARRAY to "array syntax". If "[" is just a character literal, indistinguishable in its use of array syntax, then how does one translate "[1, 2, 3]" back to "array (1, 2, 3)" like in this post: http://stackoverflow.com/questions/21500628/porting-php-5-4-to-5-3 ------------------------------------------------------------------------ [2014-02-05 20:31:54] nikic@php.net T_ARRAY represents the keyword "array", not the semantic concept of an array. The '[' will be provided like any other single-character token. ------------------------------------------------------------------------ [2014-02-05 20:09:21] bishop@php.net Description: ------------ token_get_all() identifies the start of an array using array () syntax with T_ARRAY. I expected token_get_all() to do the same for arrays using the new bracket [] syntax. Instead, token_get_all() ignores bracket style array notation altogether. This means that anyone relying on T_ARRAY parsing will not be able to see the new array syntax notation. Test script: --------------- <?php array_map( function ($string) { array_map( function ($token) { if (is_array($token)) { echo token_name($token[0]) . ', '; } }, token_get_all($string) ); echo PHP_EOL; }, array ('<?php $a = array (1);', '<?php $a = [1]') ); Expected result: ---------------- T_OPEN_TAG, T_VARIABLE, T_WHITESPACE, T_WHITESPACE, T_ARRAY, T_WHITESPACE, T_LNUMBER, T_OPEN_TAG, T_VARIABLE, T_WHITESPACE, T_WHITESPACE, T_ARRAY, T_WHITESPACE, T_LNUMBER, Actual result: -------------- T_OPEN_TAG, T_VARIABLE, T_WHITESPACE, T_WHITESPACE, T_ARRAY, T_WHITESPACE, T_LNUMBER, T_OPEN_TAG, T_VARIABLE, T_WHITESPACE, T_WHITESPACE, T_LNUMBER, ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=66649&edit=1

« previous php.bugs (#184175) next »