bug PHP-CS-Fixer#5029 SingleLineAfterImportsFixer - fix for line afte…

…r import already added using CRLF (kubawerlos) This PR was squashed before being merged into the 2.15 branch. Discussion ---------- SingleLineAfterImportsFixer - fix for line after import already added using CRLF Fixes PHP-CS-Fixer#5028 Commits ------- 5881633 SingleLineAfterImportsFixer - fix for line after import already added using CRLF
nutshellcrm · Jun 27, 2020 · dfc4f07 · dfc4f07
2 parents 2d49b15 + 5881633
commit dfc4f07
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/src/Fixer/Import/SingleLineAfterImportsFixer.php b/src/Fixer/Import/SingleLineAfterImportsFixer.php
@@ -137,6 +137,9 @@ protected function applyFix(\SplFileInfo $file, Tokens $tokens)
 
                 if ($tokens[$insertIndex]->isWhitespace()) {
                     $nextToken = $tokens[$insertIndex];
+                    if (2 === substr_count($nextToken->getContent(), "\n")) {
+                        continue;
+                    }
                     $nextMeaningfulAfterUseIndex = $tokens->getNextMeaningfulToken($insertIndex);
                     if (null !== $nextMeaningfulAfterUseIndex && $tokens[$nextMeaningfulAfterUseIndex]->isGivenKind(T_USE)) {
                         if (substr_count($nextToken->getContent(), "\n") < 2) {

diff --git a/tests/Fixer/Import/SingleLineAfterImportsFixerTest.php b/tests/Fixer/Import/SingleLineAfterImportsFixerTest.php
@@ -436,6 +436,13 @@ class Test {}
 use some\a\ClassA; use function some\a\fn_a; use const some\c;
 ',
             ],
+            [
+                str_replace("\n", "\r\n", '<?php
+use Foo;
+
+class Bar {}
+'),
+            ],
         ];
     }