Merge branch 'hotfix/24'

Close #24

CHANGELOG.md

@@ -48,6 +48,10 @@ All notable changes to this project will be documented in this file, in reverse
 
 - [#23](https://github.com/webimpress/coding-standard/pull/23) `Functions\ReturnType` - fixes recognising yoda comparison in return statement and type of returned value
 
+- [#24](https://github.com/webimpress/coding-standard/pull/24) fixes error in fixer when recognised type was invalid. Affects the following sniffs:
+    - `PHP\CorrectClassNameCase`
+    - `PHP\DisallowFqn`
+
 ## 1.0.2 - 2019-05-12
 
 ### Added

src/WebimpressCodingStandard/Sniffs/PHP/CorrectClassNameCaseSniff.php

@@ -20,6 +20,7 @@
 use function implode;
 use function in_array;
 use function ltrim;
+use function preg_match;
 use function preg_match_all;
 use function preg_quote;
 use function preg_replace;
@@ -313,6 +314,10 @@ private function checkTag(File $phpcsFile, int $stackPtr) : void
     private function getExpectedName(File $phpcsFile, string $class, int $stackPtr) : string
     {
         $suffix = strstr($class, '[');
+        if ($suffix && ! preg_match('/^(\[\])+$/', $suffix)) {
+            return $class;
+        }
+
         $class = str_replace(['[', ']'], '', $class);
 
         $imports = $this->getGlobalUses($phpcsFile);

src/WebimpressCodingStandard/Sniffs/PHP/DisallowFqnSniff.php

@@ -20,6 +20,7 @@
 use function implode;
 use function in_array;
 use function ltrim;
+use function preg_match;
 use function preg_match_all;
 use function preg_quote;
 use function preg_replace;
@@ -195,11 +196,7 @@ private function processTag(
             $localToImport = [];
             $newTypesArr = [];
             foreach ($typesArr as $name) {
-                $suffix = strstr($name, '[');
-                $name = str_replace(['[', ']'], '', $name);
-
-                $newTypesArr[] = $this->getExpectedName($phpcsFile, $stackPtr + 2, $namespace, $name, $localToImport)
-                    . $suffix;
+                $newTypesArr[] = $this->getExpectedName($phpcsFile, $stackPtr + 2, $namespace, $name, $localToImport);
             }
 
             $newTypes = implode('|', $newTypesArr);
@@ -244,30 +241,34 @@ private function getExpectedName(
             return $name;
         }
 
-        // Remove leading slash from the class name
-        $name = ltrim($name, '\\');
+        $suffix = strstr($name, '[');
+        if ($suffix && ! preg_match('/^(\[\])+$/', $suffix)) {
+            return $name;
+        }
+
+        $name = str_replace(['[', ']'], '', ltrim($name, '\\'));
 
         if (stripos($name . '\\', $namespace . '\\') === 0) {
-            return substr($name, strlen($namespace) + 1);
+            return substr($name, strlen($namespace) + 1) . $suffix;
         }
 
         $alias = $this->getAliasFromName($name);
         foreach ($this->imported['class'] ?? [] as $class) {
             // If namespace or part of it is already imported
             if (stripos($name . '\\', $class['fqn'] . '\\') === 0) {
-                return $class['name'];
+                return $class['name'] . $suffix;
             }
         }
 
         // We can't suggest anything in that case
         if (! $this->isValidClassName($phpcsFile, $stackPtr, $alias, $name)) {
-            return '\\' . $name;
+            return '\\' . $name . $suffix;
         }
 
         // We need to import it
         $toImport += $this->import('class', $name, $alias);
 
-        return $alias;
+        return $alias . $suffix;
     }
 
     private function processString(

test/Sniffs/PHP/CorrectClassNameCaseUnitTest.inc

@@ -91,4 +91,9 @@ class MyClass
     public function trav(iterable $a)
     {
     }
+
+    /**
+     * @param \Bar[]\Foo[] $param
+     */
+    abstract public function invalidTypeFormatIsNotChanged($param);
 }

test/Sniffs/PHP/CorrectClassNameCaseUnitTest.inc.fixed

@@ -91,4 +91,9 @@ class MyClass
     public function trav(iterable $a)
     {
     }
+
+    /**
+     * @param \Bar[]\Foo[] $param
+     */
+    abstract public function invalidTypeFormatIsNotChanged($param);
 }

test/Sniffs/PHP/DisallowFqnUnitTest.inc

@@ -144,4 +144,9 @@ class TheClass extends \ MyNamespace \ Hello \ ParentClass implements \ArrayAcce
             }
         };
     }
+
+    /**
+     * @param \Bar[]\Foo[] $param
+     */
+    abstract public function invalidTypeFormatIsNotChanged($param);
 }

test/Sniffs/PHP/DisallowFqnUnitTest.inc.fixed

@@ -169,4 +169,9 @@ class TheClass extends ParentClass      implements ArrayAccess, Countable
             }
         };
     }
+
+    /**
+     * @param \Bar[]\Foo[] $param
+     */
+    abstract public function invalidTypeFormatIsNotChanged($param);
 }