bug symfony#45590 [Console] Revert StringInput bc break from symfony#45088 (bobthecow)

fabpot · fabpot · commit 84f0d0e960b5 · 2022-03-01T09:25:33.000+01:00
This PR was merged into the 5.3 branch. Discussion ---------- [Console] Revert StringInput bc break from symfony#45088 | Q | A | ------------- | --- | Branch? | 5.3  | Bug fix? | yes (BC break) | New feature? | no  | Deprecations? | no  | Tickets | Refs symfony#45088 | License | MIT | Doc PR | n/a Commits ------- e978194 [Console] Revert StringInput bc break from symfony#45088
diff --git a/src/Symfony/Component/Console/Input/StringInput.php b/src/Symfony/Component/Console/Input/StringInput.php
@@ -24,7 +24,8 @@
  */
 class StringInput extends ArgvInput
 {
-    public const REGEX_STRING = '([^\s\\\\]+?)';
+    public const REGEX_STRING = '([^\s]+?)(?:\s|(?<!\\\\)"|(?<!\\\\)\'|$)';
+    public const REGEX_UNQUOTED_STRING = '([^\s\\\\]+?)';
     public const REGEX_QUOTED_STRING = '(?:"([^"\\\\]*(?:\\\\.[^"\\\\]*)*)"|\'([^\'\\\\]*(?:\\\\.[^\'\\\\]*)*)\')';
 
     /**
@@ -64,7 +65,7 @@ private function tokenize(string $input): array
                 $token .= $match[1].$match[2].stripcslashes(str_replace(['"\'', '\'"', '\'\'', '""'], '', substr($match[3], 1, -1)));
             } elseif (preg_match('/'.self::REGEX_QUOTED_STRING.'/A', $input, $match, 0, $cursor)) {
                 $token .= stripcslashes(substr($match[0], 1, -1));
-            } elseif (preg_match('/'.self::REGEX_STRING.'/A', $input, $match, 0, $cursor)) {
+            } elseif (preg_match('/'.self::REGEX_UNQUOTED_STRING.'/A', $input, $match, 0, $cursor)) {
                 $token .= $match[1];
             } else {
                 // should never happen
