icedream
/
php-language-server
1
0
Fork 0
Code Releases Activity

don't crash if foreach key isn't a variable

pull/564/head
Philip Nelson 2017-12-25 11:41:50 +11:00
parent 425b2390b5
commit 2a2143ab03
No known key found for this signature in database
GPG Key ID: 8C8235644C5E2279
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
fixtures/completion/foreach.php
View File

@ -36,5 +36,5 @@ foreach ($bar->test() as $value) {
$ $
} }
foreach ($unknownArray as $unknown) { foreach ($unknownArray as $member->access => $unknown) {
$unkno $unkno

4
src/DefinitionResolver.php
View File

@ -571,7 +571,9 @@ class DefinitionResolver
// If we get to a ForeachStatement, check the keys and values // If we get to a ForeachStatement, check the keys and values
if ($n instanceof Node\Statement\ForeachStatement) { if ($n instanceof Node\Statement\ForeachStatement) {
if ($n->foreachKey && $n->foreachKey->expression->getName() === $name) { if ($n->foreachKey instanceof Node\Expression\Variable
&& $n->foreachKey->expression->getName() === $name
) {
return $n->foreachKey; return $n->foreachKey;
} }
if ($n->foreachValue if ($n->foreachValue