Merge pull request #158 from ppi/tests-dispatch

Additional error handling for invalid modules
ppi · Feb 7, 2016 · 6ef30b9 · 6ef30b9
2 parents f2388cc + 50c2186
commit 6ef30b9
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/Module/Controller/ControllerNameParser.php b/src/Module/Controller/ControllerNameParser.php
@@ -85,6 +85,14 @@ private function getPartsFromControllerName($controller)
         list($moduleName, $controller, $action) = $parts;
         $controller = str_replace('/', '\\', $controller);
         $module     = $this->moduleManager->getModule($moduleName);
+
+        if (!is_object($module)) {
+            throw new \RuntimeException(sprintf(
+                'Unable to locate module: %s, when parsing controller: %s',
+                $moduleName, $controller
+            ));
+        }
+
         $moduleName = $module->getName();
 
         return [$module, $moduleName, $controller, $action];