Improve exception messages for mount()

7d6298d8 · Damien Walsh · Fabien Potencier · b0aa888c · 7d6298d8
Commit 7d6298d8 authored May 12, 2014 by Damien Walsh Committed by Fabien Potencier Jun 06, 2014
Show whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

src/Silex/Application.php src/Silex/Application.php +6 -0

No files found.
--- a/src/Silex/Application.php
+++ b/src/Silex/Application.php
@@ -466,11 +466,17 @@ class Application extends Container implements HttpKernelInterface, TerminableIn
     * @param ControllerCollection|ControllerProviderInterface $controllers A ControllerCollection or a ControllerProviderInterface instance
     *
     * @return Application
+     *
+     * @throws \LogicException
     */
    public function mount($prefix, $controllers)
    {
        if ($controllers instanceof ControllerProviderInterface) {
            $controllers = $controllers->connect($this);
+
+            if (!$controllers instanceof ControllerCollection) {
+                throw new \LogicException('The "connect" method of the ControllerProviderInterface must return a ControllerCollection.');
+            }
        }

        if (!$controllers instanceof ControllerCollection) {