Padawan does not autocomplete when inside a multi-line associative array

[allan-simon]

Padawan does not autocomplete when inside a multi-line associative array

[mkusher]

Hello, thanks for the bug report. This happens because of using hacked php-parser. It is unable to parse line like 'a' => $a->

[allan-simon]

hmmm ok, actually if i put it on one line , then it's able to parse it correctly

by the way, now that php7 has a AST (and a lib to access it) , would it make things easier ? (like the people from this project https://github.com/etsy/phan/ are doing)

[allan-simon]

more on the bug itself, do you think it would be hard to fix, i.e if i could try my hand on it ?

[mkusher]

@allan-simon I haven't tried it yet, but I think at least it would be faster. What about fixing this bug, right now I have not enough time and would be able to fix it only after holidays.
The problem lives here https://github.com/mkusher/padawan.php/blob/master/src/Parser/ErrorFreePhpParser.php
but if it is possible to use https://github.com/nikic/PHP-Parser/blob/master/doc/component/Error.markdown then maybe it will be easy to solve

[allan-simon]

I've added some debug in ErrorFreePhpParser, and what I can see is that most of the time the exception which get returned is

Syntax error, unexpected EOF, expecting T_STRING or T_VARIABLE or '{' or '$' on line 1

even for [ 'a' => $a-> ] (for which autocompletion works

but for

[
     'a' => $a->
]

I got :

"Syntax error, unexpected T_DOUBLE_ARROW on line 1"

[allan-simon]

oh I see what you mean, as it handles on a line per line basis , hence why it works when the beginning of the array is on the same line, and for other type of multiline statement

[golaod]

Is there a chance to fix it?

[mkusher]

@golaod there should be fix in a parser logic, I'm not sure how easy it is. Currently I'm working on https://github.com/mkusher/padawan.php/pulls