Fix complex next arg (#258)

* [DowngradePhp73] Fix complex next arg on DowngradeTrailingCommasInFunctionCallsRector * [DowngradePhp73] Fix complex next arg on DowngradeTrailingCommasInFunctionCallsRector * [DowngradePhp73] Fix complex next arg on DowngradeTrailingCommasInFunctionCallsRector
rectorphp · Dec 12, 2024 · 4a42815 · 4a42815
1 parent 3aebc81
commit 4a42815
Show file tree

Hide file tree

Showing 2 changed files with 494 additions and 3 deletions.
diff --git a/rules/DowngradePhp73/Rector/FuncCall/DowngradeTrailingCommasInFunctionCallsRector.php b/rules/DowngradePhp73/Rector/FuncCall/DowngradeTrailingCommasInFunctionCallsRector.php
@@ -93,12 +93,17 @@ public function refactor(Node $node): ?Node
         $iteration = 1;
 
         while (isset($tokens[$args[$lastArgKey]->getEndTokenPos() + $iteration])) {
-            if (trim($tokens[$args[$lastArgKey]->getEndTokenPos() + $iteration]->text) === ',') {
-                $tokens[$args[$lastArgKey]->getEndTokenPos() + $iteration]->text = '';
+            if (trim($tokens[$args[$lastArgKey]->getEndTokenPos() + $iteration]->text) === '') {
+                ++$iteration;
+                continue;
+            }
+
+            if (trim($tokens[$args[$lastArgKey]->getEndTokenPos() + $iteration]->text) !== ',') {
                 break;
             }
 
-            ++$iteration;
+            $tokens[$args[$lastArgKey]->getEndTokenPos() + $iteration]->text = '';
+            break;
         }
 
         return $node;