Skip to content

Conversation

sirbrillig
Copy link
Owner

This replaces the UndefinedVariable sniff code with UndefinedArrayVariable if the variable is an "array push shortcut" (I don't know of a better name for it): $x[] = 5;

Also, when marking such a variable, this sniff considers that variable defined at that point going forward. (This will prevent future uses of the same variable from being reported as undefined.)

Fixes #98

@sirbrillig sirbrillig self-assigned this Sep 10, 2020
@sirbrillig sirbrillig added this to the 3.0 milestone Sep 10, 2020
@sirbrillig sirbrillig marked this pull request as ready for review September 10, 2020 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Direct variable-as-array assignment marked as undefined

1 participant