[FrameworkBundle] Fix Routing\DelegatingLoader

nicolas-grekas · nicolas-grekas · commit 591c976d6d06 · 2017-09-24T15:55:14.000+02:00
diff --git a/Routing/DelegatingLoader.php b/Routing/DelegatingLoader.php
@@ -85,7 +85,7 @@ public function load($resource, $type = null)
         $this->loading = false;
 
         foreach ($collection->all() as $route) {
-            if (!$controller = $route->getDefault('_controller')) {
+            if (!is_string($controller = $route->getDefault('_controller')) || !$controller) {
                 continue;
             }
 
diff --git a/Tests/Functional/Bundle/TestBundle/Resources/config/routing.yml b/Tests/Functional/Bundle/TestBundle/Resources/config/routing.yml
@@ -44,3 +44,7 @@ fragment_home:
 fragment_inlined:
     path:     /fragment_inlined
     defaults: { _controller: TestBundle:Fragment:inlined }
+
+array_controller:
+    path:     /array_controller
+    defaults: { _controller: [ArrayController, someAction] }

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ public function load($resource, $type = null)`
`85`	`85`	`$this->loading = false;`
`86`	`86`
`87`	`87`	`foreach ($collection->all() as $route) {`
`88`		`- if (!$controller = $route->getDefault('_controller')) {`
	`88`	`+ if (!is_string($controller = $route->getDefault('_controller')) \|\| !$controller) {`
`89`	`89`	`continue;`
`90`	`90`	`}`
`91`	`91`