Fixed bug #1776 : Squiz.Scope.StaticThisUsage incorrectly looking inside anon classes

Author: gsherwood

package.xml

@@ -140,6 +140,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
   - Fixed bug #1762 : Generic.WhiteSpace.Disallow[Space/Tab]Indent not inspecting content before open tag
     -- Thanks to Juliette Reinders Folmer for the patch
   - Fixed bug #1769 : Custom "define" function triggers a warning about declaring new symbols
+  - Fixed bug #1776 : Squiz.Scope.StaticThisUsage incorrectly looking inside anon classes
  </notes>
  <contents>
   <dir name="/">

src/Standards/Squiz/Sniffs/Scope/StaticThisUsageSniff.php

@@ -61,10 +61,12 @@ public function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope)
         $end  = $tokens[$stackPtr]['scope_closer'];
 
         do {
-            $next = $phpcsFile->findNext([T_VARIABLE, T_CLOSURE], ($next + 1), $end);
+            $next = $phpcsFile->findNext([T_VARIABLE, T_CLOSURE, T_ANON_CLASS], ($next + 1), $end);
             if ($next === false) {
                 continue;
-            } else if ($tokens[$next]['code'] === T_CLOSURE) {
+            } else if ($tokens[$next]['code'] === T_CLOSURE
+                || $tokens[$next]['code'] === T_ANON_CLASS
+            ) {
                 $next = $tokens[$next]['scope_closer'];
                 continue;
             } else if (strtolower($tokens[$next]['content']) !== '$this') {

src/Standards/Squiz/Tests/Scope/StaticThisUsageUnitTest.inc

@@ -41,5 +41,17 @@ class MyClass
             echo $this->name;
         };
     }
+
+    private static function func1(array $data)
+    {
+        return new class()
+        {
+            private $data;
+
+            public function __construct(array $data)
+            {
+                $this->data = $data;
+            }
+        };
+    }
 }
-?>