minor #31900 [FrameworkBundle] Fixed replace argument of routing.loader service (yceruto)

nicolas-grekas · nicolas-grekas · commit 927b1b23f161 · 2019-06-06T19:41:38.000+02:00
This PR was merged into the 5.0-dev branch. Discussion ---------- [FrameworkBundle] Fixed replace argument of `routing.loader` service | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Fix #31702 > (1/1) OutOfBoundsException The index "2" is not in the range [0, 1]. https://github.com/symfony/symfony/blob/bd8d8a2cfd50dffb4aa07908865931fb82a9e7bf/src/Symfony/Bundle/FrameworkBundle/Routing/DelegatingLoader.php#L33 Commits ------- 5e2e79d Fixed replace argument of `routing.loader` service
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php
@@ -778,7 +778,7 @@ private function registerRouterConfiguration(array $config, ContainerBuilder $co
         $loader->load('routing.xml');
 
         if ($config['utf8']) {
-            $container->getDefinition('routing.loader')->replaceArgument(2, ['utf8' => true]);
+            $container->getDefinition('routing.loader')->replaceArgument(1, ['utf8' => true]);
         }
 
         $container->setParameter('router.resource', $config['resource']);

Original file line number	Diff line number	Diff line change
`@@ -778,7 +778,7 @@ private function registerRouterConfiguration(array $config, ContainerBuilder $co`
`778`	`778`	`$loader->load('routing.xml');`
`779`	`779`
`780`	`780`	`if ($config['utf8']) {`
`781`		`- $container->getDefinition('routing.loader')->replaceArgument(2, ['utf8' => true]);`
	`781`	`+ $container->getDefinition('routing.loader')->replaceArgument(1, ['utf8' => true]);`
`782`	`782`	`}`
`783`	`783`
`784`	`784`	`$container->setParameter('router.resource', $config['resource']);`