ingeniasoftware / luthier

src/Middleware/Request.php

Spacing +10 added lines, -10 removed lines

@@ -64,19 +64,19 @@ 
 
         // FIXME: Solve ambiguity here! POST with _method="GET" makes no sense
 
-        if(isset($_POST['_method']) && in_array(strtoupper($_POST['_method']), $validMethods , TRUE))
+        if (isset($_POST['_method']) && in_array(strtoupper($_POST['_method']), $validMethods, TRUE))
             $formMethod = strtoupper($_POST['_method']);
 
-        if(is_null($formMethod))
+        if (is_null($formMethod))
         {
             $this->requestMethod = $requestMethod;
         }
         else
         {
-            if($requestMethod == 'POST')
+            if ($requestMethod == 'POST')
                 $this->requestMethod = $formMethod;
 
-            if(!$this->CI->input->is_ajax_request() && $this->requestMethod == 'HEAD')
+            if (!$this->CI->input->is_ajax_request() && $this->requestMethod == 'HEAD')
                 $this->requestMethod = 'POST';
         }
     }
@@ -90,14 +90,14 @@ 
      */
     public function run()
     {
-        if(!$this->route)
+        if (!$this->route)
         {
-            if(is_null(Route::get404()))
+            if (is_null(Route::get404()))
                 show_404();
 
-            if(Route::get404()->controller != get_class($this->CI))
+            if (Route::get404()->controller != get_class($this->CI))
             {
-                if(ENVIRONMENT != 'production')
+                if (ENVIRONMENT != 'production')
                 {
                     show_error('The request method '.$this->requestMethod.' is not allowed to view the resource', 403, 'Forbidden method');
                 }
@@ -109,7 +109,7 @@ 
         }
         else
         {
-            if(method_exists($this->CI,$this->route->method))
+            if (method_exists($this->CI, $this->route->method))
             {
                 $this->CI->{$this->route->method}();
 
@@ -120,7 +120,7 @@ 
             }
             else
             {
-                if(ENVIRONMENT != 'production')
+                if (ENVIRONMENT != 'production')
                 {
                     show_error('The method '.$this->route->controller.'::'.$this->route->method.'() does not exists', 500, 'Method not found');
                 }

src/Core/Route.php

Spacing +110 added lines, -110 removed lines

@@ -130,17 +130,17 @@ 
      */
     public static function add($verb, $path, $attr, $hideOriginal = TRUE, $return = FALSE)
     {
-        if(!is_array($attr))
+        if (!is_array($attr))
         {
-            show_error('You must specify the route attributes as an array',500,'Route error: bad attributes');
+            show_error('You must specify the route attributes as an array', 500, 'Route error: bad attributes');
         }
 
-        if(!isset($attr['uses']))
+        if (!isset($attr['uses']))
         {
             show_error('Route requires a \'controller@method\' to be pointed and it\'s not defined in the route attributes', 500, 'Route error: missing controller');
         }
 
-        if(!preg_match('/^([a-zA-Z1-9-_]+)@([a-zA-Z1-9-_]+)$/', $attr['uses'], $parsedController) !== FALSE)
+        if (!preg_match('/^([a-zA-Z1-9-_]+)@([a-zA-Z1-9-_]+)$/', $attr['uses'], $parsedController) !== FALSE)
         {
             show_error('Route controller must be in format controller@method', 500, 'Route error: bad controller format');
         }
@@ -148,28 +148,28 @@ 
         $controller = $parsedController[1];
         $method     = $parsedController[2];
 
-        if(!is_string($path))
+        if (!is_string($path))
             show_error('Route path must be a string ', 500, 'Route error: bad route path');
 
-        if(!is_string($verb))
+        if (!is_string($verb))
             show_error('Route HTTP Verb must be a string', 500, 'Route error: bad verb type');
 
         $verb = strtoupper($verb);
 
-        if(!in_array($verb, self::$http_verbs,TRUE))
+        if (!in_array($verb, self::$http_verbs, TRUE))
         {
             $errorMsg = 'Verb "'.$verb.'" is not a valid HTTP Verb, allowed verbs:<ul>';
-            foreach( self::$http_verbs as $validVerb )
+            foreach (self::$http_verbs as $validVerb)
             {
                 $errorMsg .= '<li>'.$validVerb.'</li>';
             }
             $errorMsg .= '</ul>';
-            show_error($errorMsg,500,'Route error: unknow method');
+            show_error($errorMsg, 500, 'Route error: unknow method');
         }
 
         $route['verb'] = $verb;
 
-        $path = trim($path,'/');
+        $path = trim($path, '/');
 
         //if($path == '')
         //    $path = '/';
@@ -178,7 +178,7 @@ 
         $route['controller'] = $controller;
         $route['method']     = $method;
 
-        if(isset($attr['as']))
+        if (isset($attr['as']))
         {
             $route['name'] = $attr['as'];
         }
@@ -189,48 +189,48 @@ 
 
         $route['prefix'] = NULL;
 
-        if(!is_null(self::$prefix))
-            $route['prefix'] = self::$prefix;   # Group
-        if(isset($attr['prefix']))
+        if (!is_null(self::$prefix))
+            $route['prefix'] = self::$prefix; # Group
+        if (isset($attr['prefix']))
             $route['prefix'] = $attr['prefix']; # Specific (will overwrite group prefix)
 
         $route['namespace'] = NULL;
 
-        if(!is_null(self::$namespace))
-            $route['namespace'] = self::$namespace;   # Group
-        if(isset($attr['namespace']))
+        if (!is_null(self::$namespace))
+            $route['namespace'] = self::$namespace; # Group
+        if (isset($attr['namespace']))
             $route['namespace'] = $attr['namespace']; # Specific (will overwrite group namespace)
 
         # Removing trailing slashes
-        if(!is_null($route['prefix']))
+        if (!is_null($route['prefix']))
         {
-            $route['prefix'] = trim($route['prefix'],'/');
-            if($route['prefix'] == '')
+            $route['prefix'] = trim($route['prefix'], '/');
+            if ($route['prefix'] == '')
                 $route['prefix'] = NULL;
         }
-        if(!is_null($route['namespace']))
+        if (!is_null($route['namespace']))
         {
-            $route['namespace'] = trim($route['namespace'],'/');
-            if($route['namespace'] == '')
+            $route['namespace'] = trim($route['namespace'], '/');
+            if ($route['namespace'] == '')
                 $route['namespace'] = NULL;
         }
 
         $route['middleware'] = array();
 
-        if(isset($attr['middleware']))
+        if (isset($attr['middleware']))
         {
-            if(is_array($attr['middleware']))
+            if (is_array($attr['middleware']))
             {
-                foreach($attr['middleware'] as $middleware)
+                foreach ($attr['middleware'] as $middleware)
                     $route['middleware'][] = $middleware; # Group
             }
-            elseif( is_string($attr['middleware']))
+            elseif (is_string($attr['middleware']))
             {
                 $route['middleware'][] = $attr['middleware']; # Group
             }
             else
             {
-                show_error('Route middleware must be a string or an array',500,'Route error: bad middleware format');
+                show_error('Route middleware must be a string or an array', 500, 'Route error: bad middleware format');
             }
         }
 
@@ -240,27 +240,27 @@ 
             $compiledRoute->path => $compiledRoute->route
         ];
 
-        if($hideOriginal || self::$hideOriginals === TRUE || ($compiledRoute->path != '' && $compiledRoute->path != '/' ) )
+        if ($hideOriginal || self::$hideOriginals === TRUE || ($compiledRoute->path != '' && $compiledRoute->path != '/'))
         {
             $hiddenRoutePath      = $controller.'/'.$method;
             $hiddenRouteNamespace = '';
 
-            if(!is_null($route['namespace']))
+            if (!is_null($route['namespace']))
             {
                 $hiddenRouteNamespace = $route['namespace'].'/';
             }
 
             $hiddenRoutePath = $hiddenRouteNamespace.$hiddenRoutePath;
 
-            if($method == 'index')
+            if ($method == 'index')
             {
-                self::$hiddenRoutes[] = [ $hiddenRouteNamespace.$controller  => function(){ show_404(); }];
+                self::$hiddenRoutes[] = [$hiddenRouteNamespace.$controller  => function() { show_404(); }];
             }
 
-            self::$hiddenRoutes[] = [$hiddenRoutePath => function(){ show_404(); }];
+            self::$hiddenRoutes[] = [$hiddenRoutePath => function() { show_404(); }];
         }
 
-        if(!$return)
+        if (!$return)
         {
             self::$routes[] = (object) $route;
         }
@@ -284,7 +284,7 @@ 
      */
     public static function get($url, $attr, $hideOriginal = TRUE)
     {
-        self::add('GET', $url,$attr, $hideOriginal);
+        self::add('GET', $url, $attr, $hideOriginal);
     }
 
     /**
@@ -301,7 +301,7 @@ 
      */
     public static function post($url, $attr, $hideOriginal = TRUE)
     {
-        self::add('POST', $url,$attr, $hideOriginal);
+        self::add('POST', $url, $attr, $hideOriginal);
     }
 
     /**
@@ -318,7 +318,7 @@ 
      */
     public static function put($url, $attr, $hideOriginal = TRUE)
     {
-        self::add('PUT', $url,$attr, $hideOriginal);
+        self::add('PUT', $url, $attr, $hideOriginal);
     }
 
     /**
@@ -335,7 +335,7 @@ 
      */
     public static function patch($url, $attr, $hideOriginal = TRUE)
     {
-        self::add('PATCH', $url,$attr, $hideOriginal);
+        self::add('PATCH', $url, $attr, $hideOriginal);
     }
 
     /**
@@ -352,7 +352,7 @@ 
      */
     public static function delete($url, $attr, $hideOriginal = TRUE)
     {
-        self::add('DELETE', $url,$attr, $hideOriginal);
+        self::add('DELETE', $url, $attr, $hideOriginal);
     }
 
     /**
@@ -369,7 +369,7 @@ 
      */
     public static function any($url, $attr, $hideOriginal = TRUE)
     {
-        foreach(self::$http_verbs as $verb)
+        foreach (self::$http_verbs as $verb)
         {
             $verb = strtolower($verb);
             self::add($verb, $url, $attr, $hideOriginal);
@@ -391,10 +391,10 @@ 
      */
     public static function matches($verbs, $url, $attr, $hideOriginal = FALSE)
     {
-        if(!is_array($verbs))
+        if (!is_array($verbs))
             show_error('Route::matches() first argument must be an array of valid HTTP Verbs', 500, 'Route error: bad Route::matches() verb list');
 
-        foreach($verbs as $verb)
+        foreach ($verbs as $verb)
         {
             self::add($verb, $url, $attr, $hideOriginal);
         }
@@ -429,25 +429,25 @@ 
 
         $hideOriginal = FALSE;
 
-        if(isset($attr['namespace']))
+        if (isset($attr['namespace']))
             $base_attr['namespace']  = $attr['namespace'];
 
-        if(isset($attr['middleware']))
+        if (isset($attr['middleware']))
             $base_attr['middleware'] = $attr['middleware'];
 
-        if(isset($attr['hideOriginal']))
+        if (isset($attr['hideOriginal']))
             $hideOriginal = (bool) $attr['hideOriginal'];
 
-        if(isset($attr['prefix']))
-            $base_attr['prefix']  = $attr['prefix'];
+        if (isset($attr['prefix']))
+            $base_attr['prefix'] = $attr['prefix'];
 
         $only = array();
 
-        if(isset($attr['only']) && (is_array($attr['only']) || is_string($attr['only'])))
+        if (isset($attr['only']) && (is_array($attr['only']) || is_string($attr['only'])))
         {
-            if(is_array($attr['only']))
+            if (is_array($attr['only']))
             {
-                $only  = strtolower($attr['only']);
+                $only = strtolower($attr['only']);
             }
             else
             {
@@ -455,43 +455,43 @@ 
             }
         }
 
-        if(empty($only) || in_array('index', $only))
+        if (empty($only) || in_array('index', $only))
         {
-            $route_attr = array_merge($base_attr, ['uses' => $controller.'@index',   'as' => $name.'.index']);
+            $route_attr = array_merge($base_attr, ['uses' => $controller.'@index', 'as' => $name.'.index']);
             self::get('/', $route_attr, $hideOriginal);
         }
 
-        if(empty($only) || in_array('create', $only))
+        if (empty($only) || in_array('create', $only))
         {
             $route_attr = array_merge($base_attr, ['uses' => $controller.'@create', 'as' => $name.'.create']);
             self::get('create', $route_attr, $hideOriginal);
         }
 
-        if(empty($only) || in_array('store', $only))
+        if (empty($only) || in_array('store', $only))
         {
             $route_attr = array_merge($base_attr, ['uses' => $controller.'@store', 'as' => $name.'.store']);
             self::post('/', $route_attr, $hideOriginal);
         }
 
-        if(empty($only) || in_array('show', $only))
+        if (empty($only) || in_array('show', $only))
         {
             $route_attr = array_merge($base_attr, ['uses' => $controller.'@show', 'as' => $name.'.show']);
             self::get('{slug}', $route_attr, $hideOriginal);
         }
 
-        if(empty($only) || in_array('edit', $only))
+        if (empty($only) || in_array('edit', $only))
         {
             $route_attr = array_merge($base_attr, ['uses' => $controller.'@edit', 'as' => $name.'.edit']);
             self::get('{slug}/edit', $route_attr, $hideOriginal);
         }
 
-        if(empty($only) || in_array('update', $only))
+        if (empty($only) || in_array('update', $only))
         {
             $route_attr = array_merge($base_attr, ['uses' => $controller.'@update', 'as' => $name.'.update']);
             self::matches(['PUT', 'PATCH'], '{slug}/update', $route_attr, $hideOriginal);
         }
 
-        if(empty($only) || in_array('destroy', $only))
+        if (empty($only) || in_array('destroy', $only))
         {
             $route_attr = array_merge($base_attr, ['uses' => $controller.'@destroy', 'as' => $name.'.destroy']);
             self::delete('{slug}', $route_attr, $hideOriginal);
@@ -513,32 +513,32 @@ 
         $prefix    = NULL;
         $namespace = NULL;
 
-        if(!is_null($route->prefix))
+        if (!is_null($route->prefix))
         {
             $prefix = $route->prefix;
         }
 
-        if(!is_null($route->namespace))
+        if (!is_null($route->namespace))
         {
             $namespace = $route->namespace;
         }
 
         $path = $route->path;
 
-        if(!is_null($prefix))
+        if (!is_null($prefix))
             $path = $prefix.'/'.$path;
 
-        if(substr($path, 0, 1) == "/" && strlen($path) > 1)
-            $path = substr($path,1);
+        if (substr($path, 0, 1) == "/" && strlen($path) > 1)
+            $path = substr($path, 1);
 
         $controller = $route->controller.'/'.$route->method;
 
-        if(!is_null($namespace))
+        if (!is_null($namespace))
             $controller = $namespace.'/'.$controller;
 
         $replaces =
             [
-                '{\((.*)\):[a-zA-Z0-9-_]*}' => '($1)',   # Custom regular expression
+                '{\((.*)\):[a-zA-Z0-9-_]*}' => '($1)', # Custom regular expression
                 '{num:[a-zA-Z0-9-_]*}'      => '(:num)', # (:num) route
                 '{any:[a-zA-Z0-9-_]*}'      => '(:any)', # (:any) route
                 '{[a-zA-Z0-9-_]*}'          => '(:any)', # Everything else
@@ -546,23 +546,23 @@ 
 
         $argCount = 0;
 
-        foreach($replaces as $regex => $replace)
+        foreach ($replaces as $regex => $replace)
         {
             $path = preg_replace('/'.$regex.'/', $replace, $path, -1, $argCount);
         }
 
-        if($argCount > 0)
+        if ($argCount > 0)
         {
-            for($i = 0; $i < $argCount; $i++)
+            for ($i = 0; $i < $argCount; $i++)
             {
                 $controller .= '/$'.($i + 1);
             }
         }
 
         // Removing trailing slash (it causes 404 even if the route exists, I don't know why)
-        if(substr($path,-1) == '/')
+        if (substr($path, -1) == '/')
         {
-            $path = substr($path,0,-1);
+            $path = substr($path, 0, -1);
         }
 
         return (object) [
@@ -583,11 +583,11 @@ 
     {
         $routes = array();
 
-        foreach(self::$routes as $index => $route)
+        foreach (self::$routes as $index => $route)
         {
             $compiled = self::compileRoute($route);
 
-            if( !isset($routes[$compiled->path]) || $route->verb == 'GET' )
+            if (!isset($routes[$compiled->path]) || $route->verb == 'GET')
             {
                 $routes[$compiled->path] = $compiled->route;
             }
@@ -595,16 +595,16 @@ 
             self::$routes[$index] = (object) $route;
         }
 
-        foreach(self::$hiddenRoutes as $route)
+        foreach (self::$hiddenRoutes as $route)
         {
             $path = key($route);
             $_404 = $route[$path];
 
-            if(!isset($routes[$path]))
+            if (!isset($routes[$path]))
                 $routes[$path] = $_404;
         }
 
-        if(is_null(self::$defaultController))
+        if (is_null(self::$defaultController))
             show_error('You must specify a home route: Route::home() as default controller!', 500, 'Route error: missing default controller');
 
         $defaultController = self::$defaultController->compiled;
@@ -612,7 +612,7 @@ 
 
         $routes['default_controller'] = $defaultController;
 
-        if(is_null(self::$_404page))
+        if (is_null(self::$_404page))
         {
             $routes['404_override'] = '';
         }
@@ -639,20 +639,20 @@ 
      */
     public static function group($attr, $routes)
     {
-        if(!is_array($attr))
+        if (!is_array($attr))
             show_error('Group attribute must be a valid array');
 
-        if(!isset($attr['prefix']))
+        if (!isset($attr['prefix']))
             show_error('You must specify an prefix!');
 
         self::$prefix = $attr['prefix'];
 
-        if(isset($attr['namespace']))
+        if (isset($attr['namespace']))
         {
             self::$namespace = $attr['namespace'];
         }
 
-        if(isset($attr['hideOriginals']) && $attr['hideOriginals'] === TRUE)
+        if (isset($attr['hideOriginals']) && $attr['hideOriginals'] === TRUE)
         {
             self::$hideOriginals = TRUE;
         }
@@ -661,19 +661,19 @@ 
             self::$hideOriginals = FALSE;
         }
 
-        if(isset($attr['middleware']))
+        if (isset($attr['middleware']))
         {
-            if(is_array($attr['middleware']) || is_string($attr['middleware']))
+            if (is_array($attr['middleware']) || is_string($attr['middleware']))
             {
                 //self::$middleware = $attr['middleware'];
-                if(is_array($attr['middleware']) && !empty($attr['middleware']))
+                if (is_array($attr['middleware']) && !empty($attr['middleware']))
                 {
-                    foreach($attr['middleware'] as $middleware)
-                        self::$groupMiddleware[] = [ $attr['prefix'] => $middleware ];
+                    foreach ($attr['middleware'] as $middleware)
+                        self::$groupMiddleware[] = [$attr['prefix'] => $middleware];
                 }
                 else
                 {
-                    self::$groupMiddleware[] = [ $attr['prefix'] => $attr['middleware'] ];
+                    self::$groupMiddleware[] = [$attr['prefix'] => $attr['middleware']];
                 }
             }
             else
@@ -684,17 +684,17 @@ 
 
         # Special workarround for default group controller:
         # Probably i'll remove it.
-        if(isset($attr['default']))
+        if (isset($attr['default']))
         {
             $url = str_ireplace('@', '/', $attr['default']);
-            $uri = explode('/',$attr['default']);
+            $uri = explode('/', $attr['default']);
             $controller = $uri[0];
             $method = isset($uri[1]) ? $uri[1] : NULL;
 
-            if(self::$hideOriginals === TRUE)
-                self::$hiddenRoutes[] = [$controller => function(){ show_404(); }];
+            if (self::$hideOriginals === TRUE)
+                self::$hiddenRoutes[] = [$controller => function() { show_404(); }];
 
-            $route_attr['uses'] = $controller.( !is_null($method) ? '/'.$method : '');
+            $route_attr['uses'] = $controller.(!is_null($method) ? '/'.$method : '');
             self::get('/', $route_attr);
         }
 
@@ -727,16 +727,16 @@ 
                 'as'   => $as
             ];
 
-        if(!is_null($attr) && !is_array($attr))
-            show_error('Default controller attributes must be an array',500,'Route error: bad attribute type');
+        if (!is_null($attr) && !is_array($attr))
+            show_error('Default controller attributes must be an array', 500, 'Route error: bad attribute type');
 
-        if(!is_null($attr))
-            $routeAttr = array_merge($routeAttr,$attr);
+        if (!is_null($attr))
+            $routeAttr = array_merge($routeAttr, $attr);
 
-        if(isset($attr['prefix']))
-            show_error('Default controller may not have a prefix!',500,'Route error: prefix not allowed');
+        if (isset($attr['prefix']))
+            show_error('Default controller may not have a prefix!', 500, 'Route error: prefix not allowed');
 
-        self::$defaultController = self::$routes[] = self::add('GET', '/', ['uses' => $controller, 'as' => $as],TRUE, TRUE);
+        self::$defaultController = self::$routes[] = self::add('GET', '/', ['uses' => $controller, 'as' => $as], TRUE, TRUE);
     }
 
     /**
@@ -754,14 +754,14 @@ 
         $baseAttr['prefix']     = 'auth';
         $baseAttr['middleware'] = 'Auth';
 
-        if(!is_null($attr) && is_array($attr))
+        if (!is_null($attr) && is_array($attr))
         {
             $baseAttr = array_merge($baseAttr, $attr);
         }
 
         self::group($baseAttr, function() use($controller)
         {
-            self::matches(['get','post'], '/login',  ['uses' => $controller.'@login',  'as' => $controller.'.login']);
+            self::matches(['get', 'post'], '/login', ['uses' => $controller.'@login', 'as' => $controller.'.login']);
             self::get('/logout', ['uses' => $controller.'@logout', 'as' => $controller.'.logout']);
         });
     }
@@ -822,9 +822,9 @@ 
      */
     public static function getRouteByName($search, $args = NULL)
     {
-        foreach(self::$routes as $route)
+        foreach (self::$routes as $route)
         {
-            if($route->name == $search)
+            if ($route->name == $search)
                 return base_url(self::compileRoute($route)->path);
         }
 
@@ -845,18 +845,18 @@ 
     {
         $routes = array();
 
-        foreach(self::$routes as $route)
+        foreach (self::$routes as $route)
         {
             $routes[$route->verb][] = $route;
         }
 
-        if(empty($routes))
+        if (empty($routes))
             return FALSE;
 
-        if(is_null($requestMethod))
+        if (is_null($requestMethod))
             $requestMethod = $_SERVER['REQUEST_METHOD'];
 
-        if(!isset($routes[$requestMethod]))
+        if (!isset($routes[$requestMethod]))
             return FALSE;
 
         $routes = $routes[$requestMethod];
@@ -866,18 +866,18 @@ 
 
         $path = trim($path);
 
-        if($path == '')
+        if ($path == '')
         {
             return self::$defaultController;
         }
 
-        foreach( [$path, $path.'/index'] as $findPath )
+        foreach ([$path, $path.'/index'] as $findPath)
         {
-            foreach($routes as $route)
+            foreach ($routes as $route)
             {
                 $compiled = $route->compiled;
 
-                if($findPath == key($compiled))
+                if ($findPath == key($compiled))
                     return $route;
             }
         }