[bug] parens and staticlookup

[alexander-akait]

$var = ($var[0])::foo;
$var = ($var[0][1])::foo;
$var = ($var[0])[1]::foo;
$var = (($var[0])[1])::foo;
$var = (new Foo())::bar;

Problem exists in master

[alexander-akait]

Here other big bug

get_class($this->resource)::$wrap;

[ichiriac]

not sure yet, but seems that this bug existed before

[alexander-akait]

maybe 😞

[alexander-akait]

Need also fix:

get_class($var)::$$$$$property;

[ichiriac]

Ok, for $$$$$property I'm not enough permissive here : https://github.com/glayzzle/php-parser/blob/master/src/parser/variable.js#L154

[ichiriac]

other bugs are from : https://github.com/glayzzle/php-parser/blob/master/src/parser/variable.js#L232 and introduced here #179

I've forced the call as the bugfix request was over staticlookup with call functions, but here we are working on another branch :

simple_variable:
		T_VARIABLE			{ $$ = $1; }
	|	'$' '{' expr '}'	{ $$ = $3; }
	|	'$' simple_variable	{ $$ = zend_ast_create(ZEND_AST_VAR, $2); }
;
static_member:
		class_name T_PAAMAYIM_NEKUDOTAYIM simple_variable
			{ $$ = zend_ast_create(ZEND_AST_STATIC_PROP, $1, $3); }
	|	variable_class_name T_PAAMAYIM_NEKUDOTAYIM simple_variable
			{ $$ = zend_ast_create(ZEND_AST_STATIC_PROP, $1, $3); }
;

In principle it's fixed, but can you please provide more tests and check if it's not too permissive.

[ichiriac]

released in 3.0.0-prerelease.8