PHP 8.0 | Squiz/DisallowSizeFunctionsInLoops: allow for nullsafe obje…

…ct operator Includes unit test.
squizlabs · Sep 1, 2020 · 3a094b7 · 3a094b7
1 parent 85f3521
commit 3a094b7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Standards/Squiz/Sniffs/PHP/DisallowSizeFunctionsInLoopsSniff.php b/src/Standards/Squiz/Sniffs/PHP/DisallowSizeFunctionsInLoopsSniff.php
@@ -95,7 +95,9 @@ public function process(File $phpcsFile, $stackPtr)
                     $functionName = 'object.'.$functionName;
                 } else {
                     // Make sure it isn't a member var.
-                    if ($tokens[($i - 1)]['code'] === T_OBJECT_OPERATOR) {
+                    if ($tokens[($i - 1)]['code'] === T_OBJECT_OPERATOR
+                        || $tokens[($i - 1)]['code'] === T_NULLSAFE_OBJECT_OPERATOR
+                    ) {
                         continue;
                     }
 

diff --git a/src/Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.inc b/src/Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.inc
@@ -55,4 +55,4 @@ do {
 } while($a->count);
 
 for ($i = 0; $i < $a->count; $i++) {}
-?>
+for ($i = 0; $i < $a?->count; $i++) {}