NonPrintableCharacterFixer - fix for string in single quotes, having …

…non-breaking space, linebreak, and single quote inside
PHP-CS-Fixer · Dec 14, 2021 · c02a447 · c02a447
1 parent 2c944f8
commit c02a447
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Fixer/Basic/NonPrintableCharacterFixer.php b/src/Fixer/Basic/NonPrintableCharacterFixer.php
@@ -165,7 +165,7 @@ protected function applyFix(\SplFileInfo $file, Tokens $tokens): void
 
                 if ($swapQuotes) {
                     $content = str_replace('"', '\"', $content);
-                    $content = Preg::replace('/^\'(.*)\'$/', '"$1"', $content);
+                    $content = Preg::replace('/^\'(.*)\'$/s', '"$1"', $content);
                 }
 
                 $tokens[$index] = new Token([$token->getId(), strtr($content, $escapeSequences)]);

diff --git a/tests/Fixer/Basic/NonPrintableCharacterFixerTest.php b/tests/Fixer/Basic/NonPrintableCharacterFixerTest.php
@@ -303,6 +303,10 @@ function f(string $p)
 INPUT
                  , pack('H*', 'e2808b')),
             ],
+            [
+                "<?php \"String in single quotes, having non-breaking space: \\u{a0}, linebreak: \n, and single quote inside: ' is a dangerous mix.\";",
+                "<?php 'String in single quotes, having non-breaking space: ".pack('H*', 'c2a0').", linebreak: \n, and single quote inside: \\' is a dangerous mix.';",
+            ],
         ];
     }
 }