[Yaml] Fix parsing of unquoted strings in Parser::lexUnquotedString() to ignore spaces

Link1515 · xabbuh · commit dc8898aec40a · 2025-01-01T12:47:39.000+01:00
diff --git a/src/Symfony/Component/Yaml/Parser.php b/src/Symfony/Component/Yaml/Parser.php
@@ -1158,7 +1158,7 @@ private function lexInlineQuotedString(int &$cursor = 0): string
     private function lexUnquotedString(int &$cursor): string
     {
         $offset = $cursor;
-        $cursor += strcspn($this->currentLine, '[]{},: ', $cursor);
+        $cursor += strcspn($this->currentLine, '[]{},:', $cursor);
 
         if ($cursor === $offset) {
             throw new ParseException('Malformed unquoted YAML string.');
diff --git a/src/Symfony/Component/Yaml/Tests/ParserTest.php b/src/Symfony/Component/Yaml/Tests/ParserTest.php
@@ -1710,6 +1710,33 @@ public function testBackslashInQuotedMultiLineString()
         $this->assertSame($expected, $this->parser->parse($yaml));
     }
 
+    /**
+     * @dataProvider wrappedUnquotedStringsProvider
+     */
+    public function testWrappedUnquotedStringWithMultipleSpacesInValue(string $yaml, array $expected)
+    {
+        $this->assertSame($expected, $this->parser->parse($yaml));
+    }
+
+    public static function wrappedUnquotedStringsProvider() {
+        return [
+            'mapping' => [
+                '{ foo: bar  bar, fiz: cat      cat }',
+                [
+                    'foo' => 'bar  bar',
+                    'fiz' => 'cat      cat',
+                ]
+            ],
+            'sequence' => [
+                '[ bar  bar, cat      cat ]',
+                [
+                    'bar  bar',
+                    'cat      cat',
+                ]
+            ],
+        ];
+    }
+
     public function testParseMultiLineUnquotedString()
     {
         $yaml = <<<EOT

Original file line number	Diff line number	Diff line change
`@@ -1158,7 +1158,7 @@ private function lexInlineQuotedString(int &$cursor = 0): string`
`1158`	`1158`	`private function lexUnquotedString(int &$cursor): string`
`1159`	`1159`	`{`
`1160`	`1160`	`$offset = $cursor;`
`1161`		`- $cursor += strcspn($this->currentLine, '[]{},: ', $cursor);`
	`1161`	`+ $cursor += strcspn($this->currentLine, '[]{},:', $cursor);`
`1162`	`1162`
`1163`	`1163`	`if ($cursor === $offset) {`
`1164`	`1164`	`throw new ParseException('Malformed unquoted YAML string.');`