PSR12/CompoundNamespaceDepth: fix sniff to work with the PHP 8 identifier tokens

The existing unit tests already contain tests covering this change.

The tests also include a test which is a parse error in PHP >= 8.0. The current fix will handle this the same in both PHP < 8 as well as PHP 8.0+ and will throw the expected errors even so.

The parse error in PHP 8 has been annotated in the test case file.

Author: jrfnl

src/Standards/PSR12/Sniffs/Namespaces/CompoundNamespaceDepthSniff.php

@@ -62,6 +62,11 @@ public function process(File $phpcsFile, $stackPtr)
                 continue;
             }
 
+            if ($tokens[$i]['code'] === T_NAME_FULLY_QUALIFIED || $tokens[$i]['code'] === T_NAME_QUALIFIED) {
+                $depth += substr_count($tokens[$i]['content'], '\\');
+                continue;
+            }
+
             if ($i === $end || $tokens[$i]['code'] === T_COMMA) {
                 // End of a namespace.
                 if ($depth > $this->maxDepth) {
@@ -72,7 +77,7 @@ public function process(File $phpcsFile, $stackPtr)
 
                 $depth = 1;
             }
-        }
+        }//end for
 
     }//end process()
 

src/Standards/PSR12/Tests/Namespaces/CompoundNamespaceDepthUnitTest.inc

@@ -12,6 +12,7 @@ use Vendor\Package\SomeNamespace\{
     ClassZ,
 };
 
+// Parse error in PHP 8.0+, but will be detected correctly cross-version anyway.
 use Vendor\Package\SomeNamespace\{
     SubnamespaceOne /* comment */
     \AnotherNamespace // comment

src/Standards/PSR12/Tests/Namespaces/CompoundNamespaceDepthUnitTest.php

@@ -32,8 +32,8 @@ public function getErrorList()
     {
         return [
             10 => 1,
-            18 => 1,
-            21 => 1,
+            19 => 1,
+            22 => 1,
         ];
 
     }//end getErrorList()