Improve fully qualified class sniff to check inheritance classes too

flyeralarm · Oct 9, 2020 · 85863d6 · 85863d6
1 parent 6a5a156
commit 85863d6
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 3 deletions.
diff --git a/custom-standards/Flyeralarm/Sniffs/Classes/FullyQualifiedSniff.php b/custom-standards/Flyeralarm/Sniffs/Classes/FullyQualifiedSniff.php
@@ -13,7 +13,7 @@ class FullyQualifiedSniff implements Sniff
      */
     public function register()
     {
-        return array(T_DOUBLE_COLON, T_NEW);
+        return array(T_DOUBLE_COLON, T_NEW, T_EXTENDS);
     }
 
     /**
@@ -61,6 +61,18 @@ private function getClassCall(File $phpcsFile, $stackPtr): string
                     $classCallStart,
                     $stackPtr - $classCallStart
                 );
+
+            case T_EXTENDS:
+                $tokensAsString = $phpcsFile->getTokensAsString(
+                    $stackPtr,
+                    $phpcsFile->findEndOfStatement($stackPtr) - $stackPtr
+                );
+
+                return trim(substr(
+                    $tokensAsString,
+                    0,
+                    strpos($tokensAsString, '{')
+                ));
         }
 
         throw new RuntimeException(sprintf(

diff --git a/tests/rules/classes/allowed/FullyQualifiedSniff.php b/tests/rules/classes/allowed/FullyQualifiedSniff.php
@@ -5,8 +5,9 @@
 namespace flyeralarm\Test;
 
 use RuntimeException;
+use stdClass;
 
-class FullyQualifiedSniff
+class FullyQualifiedSniff extends stdClass
 {
     public function a()
     {

diff --git a/tests/rules/classes/allowed/NamespaceVendorLowercasePackageUpperCamelCase.php b/tests/rules/classes/allowed/NamespaceVendorLowercasePackageUpperCamelCase.php
@@ -4,6 +4,8 @@
 
 namespace flyeralarm\FooBar;
 
-class NamespaceVendorLowercasePackageUpperCamelCase extends \PHP_CodeSniffer_File
+use PHP_CodeSniffer_File;
+
+class NamespaceVendorLowercasePackageUpperCamelCase extends PHP_CodeSniffer_File
 {
 }
diff --git a/tests/rules/classes/not-allowed/FullyQualifiedSniffExtends.php b/tests/rules/classes/not-allowed/FullyQualifiedSniffExtends.php
@@ -0,0 +1,13 @@
+<?php
+
+// @expectedError Qualifier should be replaced with an import: "extends \stdClass"
+
+namespace flyeralarm\Test;
+
+class FullyQualifiedSniffExtends extends \stdClass
+{
+    public function a()
+    {
+        $a = 0;
+    }
+}