Squiz/DisallowSizeFunctionsInLoops: fix sniff to work with the PHP 8 identifier tokens

Includes extra unit tests covering the change in so far it wasn't already covered.

Author: jrfnl

src/Standards/Squiz/Sniffs/PHP/DisallowSizeFunctionsInLoopsSniff.php

@@ -67,8 +67,8 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         for ($i = ($start + 1); $i < $end; $i++) {
-            if ($tokens[$i]['code'] === T_STRING
-                && isset($this->forbiddenFunctions[$tokens[$i]['content']]) === true
+            if (($tokens[$i]['code'] === T_STRING || $tokens[$i]['code'] === T_NAME_FULLY_QUALIFIED)
+                && isset($this->forbiddenFunctions[ltrim($tokens[$i]['content'], '\\')]) === true
             ) {
                 $functionName = $tokens[$i]['content'];
 

src/Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.inc

@@ -56,3 +56,8 @@ do {
 
 for ($i = 0; $i < $a->count; $i++) {}
 for ($i = 0; $i < $a?->count; $i++) {}
+
+for ($i = 0; $i < \sizeof($this->children); $i++) {}
+while ($i < \count($array)) {}
+do {
+} while ($i < \strlen($string));

src/Standards/Squiz/Tests/PHP/DisallowSizeFunctionsInLoopsUnitTest.php

@@ -43,6 +43,9 @@ public function getErrorList()
             40 => 1,
             44 => 1,
             46 => 1,
+            60 => 1,
+            61 => 1,
+            63 => 1,
         ];
 
     }//end getErrorList()