bug #3866 SpaceAfterSemicolonFixer - loop over all tokens (SpacePossum)

This PR was squashed before being merged into the 2.12 branch (closes #3866). Discussion ---------- SpaceAfterSemicolonFixer - loop over all tokens closes #3856 Commits ------- 9675267 SpaceAfterSemicolonFixer - loop over all tokens
PHP-CS-Fixer · Jul 4, 2018 · 3a1ddb6 · 3a1ddb6
2 parents a008ddb + 9675267
commit 3a1ddb6
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 0 deletions.
diff --git a/src/Fixer/Semicolon/SpaceAfterSemicolonFixer.php b/src/Fixer/Semicolon/SpaceAfterSemicolonFixer.php
@@ -113,6 +113,7 @@ protected function applyFix(\SplFileInfo $file, Tokens $tokens)
                     )
                 ) {
                     $tokens->insertAt($index + 1, new Token([T_WHITESPACE, ' ']));
+                    ++$max;
                 }
 
                 continue;

diff --git a/tests/Fixer/Semicolon/SpaceAfterSemicolonFixerTest.php b/tests/Fixer/Semicolon/SpaceAfterSemicolonFixerTest.php
@@ -258,6 +258,30 @@ public function provideFixCases()
                     }
                 ',
             ],
+            [
+                '<?php if ($a):?>
+                    1
+                <?php endif; ?>
+                <?php if ($b):?>
+                    2
+                <?php endif; ?>
+                <?php if ($c):?>
+                    3
+                <?php endif; ?>',
+                '<?php if ($a):?>
+                    1
+                <?php endif;?>
+                <?php if ($b):?>
+                    2
+                <?php endif;?>
+                <?php if ($c):?>
+                    3
+                <?php endif;?>',
+            ],
+            [
+                '<?php echo 1; ; ; ; ; ; ; ; ;',
+                '<?php echo 1;;;;;;;;;',
+            ],
         ];
     }