Skip to content

Commit 90d7331

Browse files
committed
RequireCombinedAssignmentOperatorSniff: Fixed internal error
1 parent fe6cc55 commit 90d7331

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

SlevomatCodingStandard/Sniffs/Operators/RequireCombinedAssignmentOperatorSniff.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,12 @@ public function process(File $phpcsFile, $equalPointer): void
7878

7979
/** @var int $beforeEqualEndPointer */
8080
$beforeEqualEndPointer = TokenHelper::findPreviousEffective($phpcsFile, $equalPointer - 1);
81-
/** @var int $beforeEqualStartPointer */
8281
$beforeEqualStartPointer = IdentificatorHelper::findStartPointer($phpcsFile, $beforeEqualEndPointer);
8382

83+
if ($beforeEqualStartPointer === null) {
84+
return;
85+
}
86+
8487
$beforeEqualVariableContent = IdentificatorHelper::getContent($phpcsFile, $beforeEqualStartPointer, $beforeEqualEndPointer);
8588

8689
if ($beforeEqualVariableContent !== $variableContent) {

tests/Sniffs/Operators/data/requireCombinedAssignmentOperatorNoErrors.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,4 +27,11 @@ public function noVariableAfterAssigment($b, $c)
2727
{
2828
$a = ($b * $c);
2929
}
30+
31+
public function shortList($array)
32+
{
33+
[$a, $b, $c, $d, $e] = $array + range(0, 4);
34+
}
35+
3036
}
37+

0 commit comments

Comments
 (0)