fixup! PhpdocToParamTypeFixer - add failing cases

Author: julienfalque

src/Fixer/FunctionNotation/PhpdocToParamTypeFixer.php

@@ -252,6 +252,11 @@ protected function applyFix(\SplFileInfo $file, Tokens $tokens)
                     continue;
                 }
 
+                $byRefIndex = $tokens->getPrevMeaningfulToken($variableIndex);
+                if ($tokens[$byRefIndex]->equals('&')) {
+                    $variableIndex = $byRefIndex;
+                }
+
                 if (!('(' === $tokens[$variableIndex - 1]->getContent()) && $this->hasParamTypeHint($tokens, $variableIndex - 2)) {
                     continue;
                 }
@@ -374,44 +379,32 @@ private function fixFunctionDefinition(
         $hasCallable,
         $hasObject
     ) {
-        if (true === $hasNull) {
-            $newTokens[] = new Token([CT::T_NULLABLE_TYPE, '?']);
-        }
+        $newTokens = [];
 
         if (true === $hasVoid) {
             $newTokens[] = new Token('void');
-        }
-
-        if (true === $hasIterable && true === $hasArray) {
+        } elseif (true === $hasIterable && true === $hasArray) {
             $newTokens[] = new Token([CT::T_ARRAY_TYPEHINT, 'array']);
         } elseif (true === $hasIterable) {
             $newTokens[] = new Token([T_STRING, 'iterable']);
         } elseif (true === $hasArray) {
             $newTokens[] = new Token([CT::T_ARRAY_TYPEHINT, 'array']);
-        }
-
-        if (true === $hasString) {
+        } elseif (true === $hasString) {
             $newTokens[] = new Token([T_STRING, 'string']);
-        }
-
-        if (true === $hasInt) {
+        } elseif (true === $hasInt) {
             $newTokens[] = new Token([T_STRING, 'int']);
-        }
-
-        if (true === $hasFloat) {
+        } elseif (true === $hasFloat) {
             $newTokens[] = new Token([T_STRING, 'float']);
-        }
-
-        if (true === $hasBool) {
+        } elseif (true === $hasBool) {
             $newTokens[] = new Token([T_STRING, 'bool']);
-        }
-
-        if (true === $hasCallable) {
+        } elseif (true === $hasCallable) {
             $newTokens[] = new Token([T_CALLABLE, 'callable']);
+        } elseif (true === $hasObject) {
+            $newTokens[] = new Token([T_STRING, 'object']);
         }
 
-        if (true === $hasObject) {
-            $newTokens[] = new Token([T_STRING, 'object']);
+        if ('' !== $paramType && [] !== $newTokens) {
+            return;
         }
 
         foreach (explode('\\', $paramType) as $nsIndex => $value) {
@@ -425,6 +418,10 @@ private function fixFunctionDefinition(
             $newTokens[] = new Token([T_STRING, $value]);
         }
 
+        if (true === $hasNull) {
+            array_unshift($newTokens, new Token([CT::T_NULLABLE_TYPE, '?']));
+        }
+
         $newTokens[] = new Token([T_WHITESPACE, ' ']);
         $tokens->insertAt($index, $newTokens);
     }

tests/Fixer/FunctionNotation/PhpdocToParamTypeFixerTest.php

@@ -320,13 +320,13 @@ class Foo {
                 '<?php class Foo { /** @param Bar $bar */ public function foo($tab) { } }',
             ],
             'param by reference' => [
-                '<?php /** @param array $data */ function sort(array &$data) {}',
-                '<?php /** @param array $data */ function sort(&$data) {}',
+                '<?php /** @param array $data */ function foo(array &$data) {}',
+                '<?php /** @param array $data */ function foo(&$data) {}',
             ],
             'optional param by reference' => [
                 '<?php /** @param null|string[] $matches */ function matchAll(?array &$matches) {}',
                 '<?php /** @param null|string[] $matches */ function matchAll(&$matches) {}',
-                70200,
+                70100,
             ],
         ];
     }