maikgreubel / caribu-mvc

src/Mvc/Application.php

Indentation +42 added lines, -42 removed lines

@@ -318,21 +318,21 @@ 
      */
     public function registerController($controller, $applicationName = 'default')
     {
-    	if ( !$controller instanceof \Nkey\Caribu\Mvc\Controller\AbstractController ) {
-	        if (! class_exists($controller)) {
-	            throw new ControllerException("No such controller class {controller} found", array(
-	                'controller' => $controller
-	            ));
-	        }
-	        $c = new $controller();
-	        if (! ($c instanceof AbstractController)) {
-	            throw new ControllerException("Controller {controller} is not in application scope", array(
-	                'controller' => $controller
-	            ));
-	        }
-    	}
+        if ( !$controller instanceof \Nkey\Caribu\Mvc\Controller\AbstractController ) {
+            if (! class_exists($controller)) {
+                throw new ControllerException("No such controller class {controller} found", array(
+                    'controller' => $controller
+                ));
+            }
+            $c = new $controller();
+            if (! ($c instanceof AbstractController)) {
+                throw new ControllerException("Controller {controller} is not in application scope", array(
+                    'controller' => $controller
+                ));
+            }
+        }
         else {
-        	$c = $controller;
+            $c = $controller;
         }
         $settings = $c->getControllerSettings();
         $this->controllers[$applicationName][$settings->getControllerSimpleName()] = $settings;
@@ -375,35 +375,35 @@ 
         ));
         
         if ( null != $this->router && $this->router->hasRoute($action) ) {
-        	$controllerInstance = $this->router->route($action, $request);
-        	$action = $request->getAction();
+            $controllerInstance = $this->router->route($action, $request);
+            $action = $request->getAction();
         }
         else {
-	        if (! isset($this->controllers[$applicationName][$controller])) {
-	            $this->getLogger()->error("[{remote}] No such controller {controller}", array(
-	                'remote' => $request->getRemoteHost(),
-	                'controller' => $controller
-	            ));
-	            $controller = 'Error';
-	            $action = 'error';
-	        }
+            if (! isset($this->controllers[$applicationName][$controller])) {
+                $this->getLogger()->error("[{remote}] No such controller {controller}", array(
+                    'remote' => $request->getRemoteHost(),
+                    'controller' => $controller
+                ));
+                $controller = 'Error';
+                $action = 'error';
+            }
 	        
-	        $controllerInstance = $this->controllers[$applicationName][$controller];
-	        assert($controllerInstance instanceof AbstractController);
-	        if (! $controllerInstance->hasAction($action)) {
-	            $this->getLogger()->error("[{remote}] No such action {action}", array(
-	                'remote' => $request->getRemoteHost(),
-	                'action' => $action
-	            ));
-	            $controllerInstance = $this->controllers[$applicationName]['Error'];
-	            $action = 'error';
-	        }
+            $controllerInstance = $this->controllers[$applicationName][$controller];
+            assert($controllerInstance instanceof AbstractController);
+            if (! $controllerInstance->hasAction($action)) {
+                $this->getLogger()->error("[{remote}] No such action {action}", array(
+                    'remote' => $request->getRemoteHost(),
+                    'action' => $action
+                ));
+                $controllerInstance = $this->controllers[$applicationName]['Error'];
+                $action = 'error';
+            }
 	        
-	        $this->getLogger()->debug("[{remote}] Routing request to {controller}:{action}", array(
-	            'remote' => $request->getRemoteHost(),
-	            'controller' => $controller,
-	            'action' => $action
-	        ));
+            $this->getLogger()->debug("[{remote}] Routing request to {controller}:{action}", array(
+                'remote' => $request->getRemoteHost(),
+                'controller' => $controller,
+                'action' => $action
+            ));
         }
         
         $view = $this->getViewBestMatch($request, $applicationName);
@@ -467,9 +467,9 @@ 
      */
     public function registerRouter(AbstractRouter $router)
     {
-    	$this->router = $router;
-    	$this->router->setApplication($this);
-    	return $this;
+        $this->router = $router;
+        $this->router->setApplication($this);
+        return $this;
     }
 
     /**

src/Mvc/Util/RequestParser.php

Indentation +4 added lines, -4 removed lines

@@ -51,12 +51,12 @@ 
      * @return array Parsed parts for later usage
      */
     private static function parseUri(Request &$request,
-    		$uri, $defaultController, $defaultAction)
+            $uri, $defaultController, $defaultAction)
     {
         // All beyond the context prefix is our application request uri
         $contextUri = $uri;
         if (null != $request->getContextPrefix() && '/' != $request->getContextPrefix()) {
-        	$contextUri = str_replace($request->getContextPrefix(), '', $uri);
+            $contextUri = str_replace($request->getContextPrefix(), '', $uri);
         }
         
         // Split parts
@@ -102,10 +102,10 @@ 
      *            The destination parameter name
      */
     private static function parseElement(Request &$req,
-    		$serverVars, $elementName, $paramName)
+            $serverVars, $elementName, $paramName)
     {
         if (isset($serverVars[$elementName])) {
-        	$req->setParam( $paramName, $serverVars[$elementName] );
+            $req->setParam( $paramName, $serverVars[$elementName] );
         }
     }
 

src/Mvc/Controller/Request.php

Indentation +3 added lines, -3 removed lines

@@ -181,7 +181,7 @@ 
      */
     public function setController($controller)
     {
-    	$this->controller = strval($controller);
+        $this->controller = strval($controller);
     }
 
     /**
@@ -201,7 +201,7 @@ 
      */
     public function setAction($action)
     {
-    	$this->action = $action;
+        $this->action = $action;
     }
 
     /**
@@ -231,7 +231,7 @@ 
      */
     public function setContextPrefix($prefix)
     {
-    	$this->contextPrefix = strval($prefix);
+        $this->contextPrefix = strval($prefix);
     }
 
     /**

src/Mvc/Router/AbstractRouter.php

Indentation +66 added lines, -66 removed lines

@@ -7,77 +7,77 @@
 
 abstract class AbstractRouter {
 	
-	/**
-	 * Application instance
-	 * 
-	 * @var Application
-	 */
-	private $application;
+    /**
+     * Application instance
+     * 
+     * @var Application
+     */
+    private $application;
 	
-	public function setApplication(Application $application)
-	{
-		$this->application = $application;
-		foreach($this->routes as $routeName => $controller) {
-			$this->application->registerController($controller, $routeName);
-		}
-	}
+    public function setApplication(Application $application)
+    {
+        $this->application = $application;
+        foreach($this->routes as $routeName => $controller) {
+            $this->application->registerController($controller, $routeName);
+        }
+    }
 	
-	/**
-	 * @var array
-	 */
-	private $routes;
+    /**
+     * @var array
+     */
+    private $routes;
 	
-	public function addRoute(string $name, AbstractController $controller)
-	{
-		$this->routes[$name] = $controller;
-	}
+    public function addRoute(string $name, AbstractController $controller)
+    {
+        $this->routes[$name] = $controller;
+    }
 	
-	/**
-	 * Checks wether a route exists
-	 * 
-	 * @param string $name
-	 * @return bool
-	 */
-	public function hasRoute(string $name)
-	{
-		return key_exists($name, $this->routes);
-	}
+    /**
+     * Checks wether a route exists
+     * 
+     * @param string $name
+     * @return bool
+     */
+    public function hasRoute(string $name)
+    {
+        return key_exists($name, $this->routes);
+    }
 	
-	/**
-	 * 
-	 * @param string $name
-	 * @throws RouterException
-	 * @return AbstractController
-	 */
-	private function getRoute(string $name)
-	{
-		if(!$this->hasRoute($name)) {
-			throw new RouterException("Router {$router} is not registered");
-		}
+    /**
+     * 
+     * @param string $name
+     * @throws RouterException
+     * @return AbstractController
+     */
+    private function getRoute(string $name)
+    {
+        if(!$this->hasRoute($name)) {
+            throw new RouterException("Router {$router} is not registered");
+        }
 		
-		return $this->routes[$name];
-	}
+        return $this->routes[$name];
+    }
 	
-	/**
-	 * Route the existing request into a new controller
-	 * 
-	 * @param string $name The name of route
-	 * @param Request $request The existing request instance
-	 * @return \Nkey\Caribu\Mvc\Controller\AbstractController
-	 */
-	public function route(string $name, Request $request)
-	{
-		$parts = \explode('/', $request->getOrigin());
-		$found = false;
-		for($i = 0; $i < count($parts); $i++) {
-			if($parts[$i] === $name && isset($parts[$i+1])) {
-				$request->setAction($parts[$i+1]);
-				$found = true;
-			}
-		}
-		if(!$found) {
-			$request->setAction("index");
-		}
-		return $this->getRoute($name);
-	}
+    /**
+     * Route the existing request into a new controller
+     * 
+     * @param string $name The name of route
+     * @param Request $request The existing request instance
+     * @return \Nkey\Caribu\Mvc\Controller\AbstractController
+     */
+    public function route(string $name, Request $request)
+    {
+        $parts = \explode('/', $request->getOrigin());
+        $found = false;
+        for($i = 0; $i < count($parts); $i++) {
+            if($parts[$i] === $name && isset($parts[$i+1])) {
+                $request->setAction($parts[$i+1]);
+                $found = true;
+            }
+        }
+        if(!$found) {
+            $request->setAction("index");
+        }
+        return $this->getRoute($name);
+    }
 }
\ No newline at end of file