[Routing] Fix simple parameter mappings in routes

nicolas-grekas · nicolas-grekas · commit f04a74198d0c · 2025-12-16T20:27:23.000Z
diff --git a/src/Symfony/Component/Routing/Route.php b/src/Symfony/Component/Routing/Route.php
@@ -426,7 +426,7 @@ private function extractInlineDefaultsAndRequirements(string $pattern): string
                 $this->setRequirement($m[2], substr($m[6], 1, -1));
             }
             if (isset($m[4][0])) {
-                $mapping[$m[2]] = isset($m[5][0]) ? [$m[4], substr($m[5], 1)] : $mapping[$m[2]] = [$m[4], $m[2]];
+                $mapping[$m[2]] = isset($m[5][0]) ? [$m[4], substr($m[5], 1)] : $m[4];
             }
 
             return '{'.$m[1].$m[2].'}';
diff --git a/src/Symfony/Component/Routing/Tests/Matcher/UrlMatcherTest.php b/src/Symfony/Component/Routing/Tests/Matcher/UrlMatcherTest.php
@@ -1046,10 +1046,7 @@ public function testMapping()
             '_route' => 'a',
             'slug' => 'vienna-2024',
             '_route_mapping' => [
-                'slug' => [
-                    'conference',
-                    'slug',
-                ],
+                'slug' => 'conference',
             ],
         ];
         $this->assertEquals($expected, $matcher->match('/conference/vienna-2024'));

Original file line number	Diff line number	Diff line change
`@@ -426,7 +426,7 @@ private function extractInlineDefaultsAndRequirements(string $pattern): string`
`426`	`426`	`$this->setRequirement($m[2], substr($m[6], 1, -1));`
`427`	`427`	`}`
`428`	`428`	`if (isset($m[4][0])) {`
`429`		`- $mapping[$m[2]] = isset($m[5][0]) ? [$m[4], substr($m[5], 1)] : $mapping[$m[2]] = [$m[4], $m[2]];`
	`429`	`+ $mapping[$m[2]] = isset($m[5][0]) ? [$m[4], substr($m[5], 1)] : $m[4];`
`430`	`430`	`}`
`431`	`431`
`432`	`432`	`return '{'.$m[1].$m[2].'}';`