hnrazevedo / Router

src/Router.php

Indentation +11 added lines, -11 removed lines

@@ -44,20 +44,20 @@ 
     {
         $this->lastReturn = null;
         
-		$url = (substr($url,0,1) !=='/' and strlen($url) > 0) ? "/{$url}" : $url;
+        $url = (substr($url,0,1) !=='/' and strlen($url) > 0) ? "/{$url}" : $url;
 
-    	foreach($this->routers as $key => $value){
-    		if( md5($this->prefix . $value['url'] . $value['protocol'] ) === md5( $url . $protocol ) ){
+        foreach($this->routers as $key => $value){
+            if( md5($this->prefix . $value['url'] . $value['protocol'] ) === md5( $url . $protocol ) ){
                 throw new Exception("There is already a route with the url {$url} and with the {$protocol} protocol configured.");
             }
         }
         
         $this->checkTypeRole($walking);
 
-		$route = [
-			'url' => $this->prefix.$url,
-			'role' => $walking,
-			'protocol' => $protocol,
+        $route = [
+            'url' => $this->prefix.$url,
+            'role' => $walking,
+            'protocol' => $protocol,
             'filters' => null,
             'where' => null,
             'before' => null,
@@ -65,9 +65,9 @@ 
             'after' => null,
             'afterAll' => $this->afterAll,
             'group' => self::getInstance()->group
-		];
+        ];
 
-		$this->routers[] = $route;		
+        $this->routers[] = $route;		
         
         return self::getInstance();
     }
@@ -145,7 +145,7 @@ 
 
         $instance->getInstance()->byName($routeName);
 
-		$currentProtocol = $instance->getInstance()->getProtocol();
+        $currentProtocol = $instance->getInstance()->getProtocol();
 
         foreach(array_reverse($instance->getInstance()->routers) as $r => $route){
 
@@ -178,7 +178,7 @@ 
         
         $instance->getInstance()->currentRoute = null;
 
-	    throw new Exception('Page not found.',404);
+        throw new Exception('Page not found.',404);
     }
 
     public function filter($filters): Router

Spacing +36 added lines, -36 removed lines

@@ -4,7 +4,7 @@ 
 
 use Exception;
 
-class Router{
+class Router {
     use Helper, CheckTrait, CheckWhere, DefinitionsTrait;
 
     private static $instance = null;
@@ -25,7 +25,7 @@ 
 
     public static function create(): Router
     {
-        if(!self::getInstance()->instanced){
+        if (!self::getInstance()->instanced) {
             self::getInstance()->checkConfig();
             self::getInstance()->import(ROUTER_CONFIG['path']);
             self::getInstance()->instanced = true;
@@ -40,14 +40,14 @@ 
         return self::$instance;
     }
 
-    public function set($url ,$walking , string $protocol): Router
+    public function set($url, $walking, string $protocol): Router
     {
         $this->lastReturn = null;
         
-		$url = (substr($url,0,1) !=='/' and strlen($url) > 0) ? "/{$url}" : $url;
+		$url = (substr($url, 0, 1) !== '/' and strlen($url) > 0) ? "/{$url}" : $url;
 
-    	foreach($this->routers as $key => $value){
-    		if( md5($this->prefix . $value['url'] . $value['protocol'] ) === md5( $url . $protocol ) ){
+    	foreach ($this->routers as $key => $value) {
+    		if (md5($this->prefix.$value['url'].$value['protocol']) === md5($url.$protocol)) {
                 throw new Exception("There is already a route with the url {$url} and with the {$protocol} protocol configured.");
             }
         }
@@ -72,9 +72,9 @@ 
         return self::getInstance();
     }
 
-    public static function group(string $prefix,$callback): Router
+    public static function group(string $prefix, $callback): Router
     {
-        self::getInstance()->prefix = (substr($prefix,0,1) !== '/') ? "/{$prefix}" : $prefix;
+        self::getInstance()->prefix = (substr($prefix, 0, 1) !== '/') ? "/{$prefix}" : $prefix;
         self::getInstance()->group = sha1(date('d/m/Y h:m:i'));
         $callback();
         self::getInstance()->group = null;
@@ -85,7 +85,7 @@ 
 
     public static function where(): Router
     {
-        if(self::getInstance()->lastReturn){
+        if (self::getInstance()->lastReturn) {
             throw new Exception("It is not possible to define parameter tests for groups of routes.");
         }
 
@@ -96,7 +96,7 @@ 
 
     public static function name(string $name): Router
     {
-        if(self::getInstance()->lastReturn){
+        if (self::getInstance()->lastReturn) {
             throw new Exception("There is no reason to call a {$name} route group.");
         }
 
@@ -104,11 +104,11 @@ 
 
         self::getInstance()->hasRouteName($name);
 
-        if(in_array($name,self::getInstance()->beforeExcepts)){
+        if (in_array($name, self::getInstance()->beforeExcepts)) {
             $currentRoute['beforeAll'] = null;
         }
 
-        if(in_array($name,self::getInstance()->afterExcepts)){
+        if (in_array($name, self::getInstance()->afterExcepts)) {
             $currentRoute['afterAll'] = null;
         }
 
@@ -121,15 +121,15 @@ 
 
     private function byName(?string $routName)
     {
-        if(!is_null($routName)){
+        if (!is_null($routName)) {
             $currentProtocol = $this->getProtocol();
 
             $this->checkName($routName);
     
             $route = $this->routers[$routName];
     
-            if(!$this->checkProtocol($route['protocol'], $currentProtocol)){
-                throw new Exception('Page not found.',404);
+            if (!$this->checkProtocol($route['protocol'], $currentProtocol)) {
+                throw new Exception('Page not found.', 404);
             }
     
             $this->checkFiltering($route);
@@ -147,11 +147,11 @@ 
 
 		$currentProtocol = $instance->getInstance()->getProtocol();
 
-        foreach(array_reverse($instance->getInstance()->routers) as $r => $route){
+        foreach (array_reverse($instance->getInstance()->routers) as $r => $route) {
 
             $instance->getInstance()->currentRoute = $route;
 
-            if(!$instance->getInstance()->checkProtocol($route['protocol'], $currentProtocol)){
+            if (!$instance->getInstance()->checkProtocol($route['protocol'], $currentProtocol)) {
                 continue;
             }
 
@@ -161,11 +161,11 @@ 
 
 
             $routs = $instance->getInstance()->explodeRoutes(
-                (substr($route['url'],strlen($route['url'])-1,1) === '/') , $route['url'],
-                (substr($_SERVER['REQUEST_URI'],strlen($_SERVER['REQUEST_URI'])-1,1) === '/') , $_SERVER['REQUEST_URI']
+                (substr($route['url'], strlen($route['url'])-1, 1) === '/'), $route['url'],
+                (substr($_SERVER['REQUEST_URI'], strlen($_SERVER['REQUEST_URI'])-1, 1) === '/'), $_SERVER['REQUEST_URI']
             );
 
-            if(!$instance->getInstance()->checkToHiking($route, $routs['routeRequest'], $routs['routeLoop'])){
+            if (!$instance->getInstance()->checkToHiking($route, $routs['routeRequest'], $routs['routeLoop'])) {
                 continue;
             }
 
@@ -178,18 +178,18 @@ 
         
         $instance->getInstance()->currentRoute = null;
 
-	    throw new Exception('Page not found.',404);
+	    throw new Exception('Page not found.', 404);
     }
 
     public function filter($filters): Router
     {
-        if($this->lastReturn !== null){
+        if ($this->lastReturn !== null) {
             $currentGroup = end($this->routers)['group'];
 
             foreach ($this->routers as $key => $value) {
 
-                if($value['group'] === $currentGroup){
-                    $currentRoute = $this->addFilter($this->routers[$key],$filters);
+                if ($value['group'] === $currentGroup) {
+                    $currentRoute = $this->addFilter($this->routers[$key], $filters);
                     $this->routers[$key] = $currentRoute;
                 }
 
@@ -198,13 +198,13 @@ 
             return $this;
         }
         
-        $this->routers[count($this->routers)-1] = $this->addFilter(end($this->routers),$filters);
+        $this->routers[count($this->routers)-1] = $this->addFilter(end($this->routers), $filters);
         return $this;
     }
  
     public function before($walking): Router
     {
-        return $this->setOnRoute($walking,'before');
+        return $this->setOnRoute($walking, 'before');
     }
 
     public static function beforeAll($walking, $except = null): Router
@@ -212,12 +212,12 @@ 
         $excepts = is_array($except) ? $except : [$except];
         self::getInstance()->beforeExcepts = $excepts;
         self::getInstance()->beforeAll = $walking;
-        return self::getInstance()->setOnRoutes($walking,'beforeAll',$excepts);
+        return self::getInstance()->setOnRoutes($walking, 'beforeAll', $excepts);
     }
 
     public function after($walking): Router
     {
-        return $this->setOnRoute($walking,'after');
+        return $this->setOnRoute($walking, 'after');
     }
 
     public static function afterAll($walking, $except = null): Router
@@ -225,17 +225,17 @@ 
         $excepts = is_array($except) ? $except : [$except];
         self::getInstance()->afterExcepts = $excepts;
         self::getInstance()->afterAll = $walking;
-        return self::getInstance()->setOnRoutes($walking,'afterAll',$excepts);
+        return self::getInstance()->setOnRoutes($walking, 'afterAll', $excepts);
     }
 
     private function setOnRoute($walking, string $state): Router
     {
-        if($this->lastReturn !== null){
+        if ($this->lastReturn !== null) {
             $currentGroup = end($this->routers)['group'];
 
             foreach ($this->routers as $key => $value) {
 
-                if($value['group'] === $currentGroup){
+                if ($value['group'] === $currentGroup) {
                     $this->routers[$key][$state] = $walking;
                 }
 
@@ -249,8 +249,8 @@ 
 
     private function setOnRoutes($walking, string $state, array $excepts): Router
     {
-        foreach($this->routers as $r => $route){
-            if(!in_array($this->routers[$r]['name'],$excepts)){
+        foreach ($this->routers as $r => $route) {
+            if (!in_array($this->routers[$r]['name'], $excepts)) {
                 $this->routers[$r][$state] = $walking;
             }
         }
@@ -259,18 +259,18 @@ 
 
     public static function addFilter(array $route, $filter): array
     {
-        if(is_null($route['filters'])){
+        if (is_null($route['filters'])) {
             $route['filters'] = $filter;
             return $route;
         }
 
         $filters = (is_array($filter)) ? $filter : [0 => $filter];
 
-        if(is_array($route['filters'])){
+        if (is_array($route['filters'])) {
             foreach ($route['filters'] as $key => $value) {
                 $filters[] = $value;
             }
-        }else{
+        }else {
             $filters[] = $route['filters'];
         }
 

src/CheckTrait.php

Spacing +18 added lines, -18 removed lines

@@ -4,7 +4,7 @@ 
 
 use Exception;
 
-trait CheckTrait{
+trait CheckTrait {
     use FilterTrait, CheckWhere;
 
     protected function checkProtocol(string $expected, string $current): bool
@@ -12,21 +12,21 @@ 
         return (strtoupper($expected) === strtoupper($current));
     }
 
-    protected function checkName(string $routeName){
-        if(!array_key_exists($routeName,$this->routers)){
+    protected function checkName(string $routeName) {
+        if (!array_key_exists($routeName, $this->routers)) {
             throw new Exception('Page not found.', 404);
         }
     }
 
-    protected function checkTypeRole($role){
-        if(!is_string($role) && @get_class($role) !== 'Closure' ){
+    protected function checkTypeRole($role) {
+        if (!is_string($role) && @get_class($role) !== 'Closure') {
             throw new Exception("Improperly defined route track.");
         }
     }
 
     protected function checkConfig()
     {
-        if(!defined('ROUTER_CONFIG')){
+        if (!defined('ROUTER_CONFIG')) {
             throw new Exception("Information for loading routes has not been defined.");
         }
     }
@@ -38,12 +38,12 @@ 
 
     protected function checkParameters(array $routeLoop, array $routeRequest): bool
     {
-        foreach($routeLoop as $rr => $param){
-            if( (substr($param,0,1) === '{') ){
-                $_GET[ substr($param,1,strlen($param)-2) ] = $routeRequest[$rr];    
+        foreach ($routeLoop as $rr => $param) {
+            if ((substr($param, 0, 1) === '{')) {
+                $_GET[substr($param, 1, strlen($param)-2)] = $routeRequest[$rr];    
             }
     
-            if($this->checkParameter($param, $routeRequest[$rr])){
+            if ($this->checkParameter($param, $routeRequest[$rr])) {
                 return false;
             }
         }
@@ -52,22 +52,22 @@ 
 
     protected function checkParameter(string $routeLoop, string $routeRequest)
     {
-        return !( substr($routeLoop,0,1) === '{' ) and $routeLoop !== $routeRequest;
+        return !(substr($routeLoop, 0, 1) === '{') and $routeLoop !== $routeRequest;
     }
 
     protected function checkRole()
     {
-        if(!array_key_exists('role', $this->getData()['POST'])){
+        if (!array_key_exists('role', $this->getData()['POST'])) {
             throw new Exception('O servidor não conseguiu identificar a finalidade deste formulário.');
         }
     }
 
     protected function hasProtocol(array $route, string $currentProtocol)
     {
-        $protocols = ( is_array($route['protocol']) ) ? $route['protocol'] : [ $route['protocol'] ];
+        $protocols = (is_array($route['protocol'])) ? $route['protocol'] : [$route['protocol']];
 
-        foreach($protocols as $protocol){
-            if(strtoupper($protocol) !== strtoupper($currentProtocol)){
+        foreach ($protocols as $protocol) {
+            if (strtoupper($protocol) !== strtoupper($currentProtocol)) {
                 continue;
             }
         }
@@ -75,9 +75,9 @@ 
 
     protected function checkToHiking($route, $routeRequest, $routeLoop): bool
     {
-        if($this->checkNumparams($routeLoop, $routeRequest) || 
+        if ($this->checkNumparams($routeLoop, $routeRequest) || 
             !$this->checkParameters($routeLoop, $routeRequest) ||
-            !$this->checkWhere($route, $routeRequest)){
+            !$this->checkWhere($route, $routeRequest)) {
                 return false;
         }
         return true;
@@ -85,7 +85,7 @@ 
 
     protected function hasRouteName(string $name): void
     {
-        if(array_key_exists($name, $this->routers)){
+        if (array_key_exists($name, $this->routers)) {
             throw new Exception("There is already a route with the name {$name} configured.");
         }
     }