Add fully qualified class sniff

Author: Michael Nozdrevatykh

custom-standards/Flyeralarm/Sniffs/Classes/FullyQualifiedSniff.php

@@ -0,0 +1,65 @@
+<?php
+
+namespace Flyeralarm\CodingGuidelines\Flyeralarm\Sniffs\Classes;
+
+use PHP_CodeSniffer\Sniffs\Sniff;
+use PHP_CodeSniffer\Files\File;
+use RuntimeException;
+
+class FullyQualifiedSniff implements Sniff
+{
+    /**
+     * @return array
+     */
+    public function register()
+    {
+        return array(T_DOUBLE_COLON, T_NEW);
+    }
+
+    /**
+     * @param File $phpcsFile
+     * @param int $stackPtr
+     * @return void
+     */
+    public function process(File $phpcsFile, $stackPtr)
+    {
+        $classCall = $this->getClassCall($phpcsFile, $stackPtr);
+
+        if (strpos($classCall, '\\') === false) {
+            return;
+        }
+
+        $phpcsFile->addError(
+            'Qualifier should be replaced with an import: "%s"',
+            $stackPtr,
+            'Found',
+            [$classCall]
+        );
+    }
+
+    private function getClassCall(File $phpcsFile, $stackPtr): string
+    {
+        $tokens = $phpcsFile->getTokens();
+
+        switch ($tokens[$stackPtr]['code']) {
+            case T_NEW:
+                return $phpcsFile->getTokensAsString(
+                    $stackPtr,
+                    $phpcsFile->findEndOfStatement($stackPtr) - $stackPtr
+                );
+
+            case T_DOUBLE_COLON:
+                $classCallStart = $phpcsFile->findStartOfStatement($stackPtr);
+
+                return $phpcsFile->getTokensAsString(
+                    $classCallStart,
+                    $stackPtr - $classCallStart
+                );
+        }
+
+        throw new RuntimeException(sprintf(
+            'Unknown token type: "%s"',
+            $tokens[$stackPtr]['type']
+        ));
+    }
+}

ruleset.xml

@@ -14,4 +14,5 @@
     <rule ref="./custom-standards/Flyeralarm/Sniffs/Docblock/ExpectedExceptionMessageSniff.php"/>
     <rule ref="./custom-standards/Flyeralarm/Sniffs/Variable/LowerCamelCaseSniff.php"/>
     <rule ref="./custom-standards/Flyeralarm/Sniffs/ControlStructures/YodaSniff.php"/>
+    <rule ref="./custom-standards/Flyeralarm/Sniffs/Classes/FullyQualifiedSniff.php"/>
 </ruleset>

tests/rules/classes/allowed/FullyQualifiedSniff.php

@@ -0,0 +1,16 @@
+<?php
+
+// @expectedPass
+
+namespace flyeralarm\Test;
+
+use RuntimeException;
+
+class FullyQualifiedSniff
+{
+    public function a()
+    {
+        $className = RuntimeException::class;
+        $a = new RuntimeException();
+    }
+}

tests/rules/classes/not-allowed/FullyQualifiedSniff.php

@@ -0,0 +1,13 @@
+<?php
+
+// @expectedError Qualifier should be replaced with an import: "new \RuntimeException()"
+
+namespace flyeralarm\Test;
+
+class FullyQualifiedSniff
+{
+    public function a()
+    {
+        $a = new \RuntimeException();
+    }
+}

tests/rules/classes/not-allowed/FullyQualifiedSniffStatic.php

@@ -0,0 +1,13 @@
+<?php
+
+// @expectedError Qualifier should be replaced with an import: "$a = \RuntimeException"
+
+namespace flyeralarm\Test;
+
+class FullyQualifiedSniffStatic
+{
+    public function a()
+    {
+        $a = \RuntimeException::class;
+    }
+}