hnrazevedo / Router

src/Router.php

Braces +1 added lines, -1 removed lines

@@ -217,7 +217,7 @@
             foreach ($route['filters'] as $key => $value) {
                 $filters[] = $value;
             }
-        }else{
+        } else{
             $filters[] = $route['filters'];
         }
 

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 +28 added lines, -28 removed lines

@@ -4,7 +4,7 @@ 
 
 use Exception;
 
-class Router{
+class Router {
     use Helper, CheckTrait, CheckWhere, DefinitionsTrait, ExtraJobsTrait;
 
     private static $instance = null;
@@ -25,7 +25,7 @@ 
 
     public static function create(): Router
     {
-        if(!self::getInstance()->instanced){
+        if (!self::getInstance()->instanced) {
             self::getInstance()->instanced = true;
         }
         
@@ -38,14 +38,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.");
             }
         }
@@ -70,9 +70,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;
@@ -83,7 +83,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.");
         }
 
@@ -94,7 +94,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.");
         }
 
@@ -102,11 +102,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;
         }
 
@@ -119,15 +119,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);
@@ -145,11 +145,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;
             }
 
@@ -159,11 +159,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;
             }
 
@@ -176,18 +176,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;
                 }
 
@@ -196,24 +196,24 @@ 
             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 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'];
         }
 

examples/index.php

Spacing +2 added lines, -2 removed lines

@@ -9,7 +9,7 @@ 
 
 /* NOTE: in case of error an exception is thrown */
 
-try{
+try {
     
     Router::dispatch();
 
@@ -20,7 +20,7 @@ 
     /* Return current action route */
     $action = Router::currentRouteAction();
 
-}catch(Exception $er){
+}catch (Exception $er) {
 
     die("Code Error: {$er->getCode()}, Line: {$er->getLine()}, File: {$er->getFile()}, Message: {$er->getMessage()}.");
 

Braces +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
     /* Return current action route */
     $action = Router::currentRouteAction();
 
-}catch(Exception $er){
+} catch(Exception $er){
 
     die("Code Error: {$er->getCode()}, Line: {$er->getLine()}, File: {$er->getFile()}, Message: {$er->getMessage()}.");
 

examples/Controllers/User.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 use HnrAzevedo\Router\Controller;
 
-class User extends Controller{
+class User extends Controller {
 
     public function requereLogin($username, $password): void
     {

examples/Filters/User.php

Spacing +4 added lines, -4 removed lines

@@ -4,15 +4,15 @@
 
 use HnrAzevedo\Filter\Filter as HnrFilter;
 
-class User extends HnrFilter{
+class User extends HnrFilter {
 
     public function user_in(): bool
     {
-        $this->addMessage('user_in','User required to be logged in.');
+        $this->addMessage('user_in', 'User required to be logged in.');
 
-        $this->addTreat('user_in','report_notLogged');
+        $this->addTreat('user_in', 'report_notLogged');
 
-        return (array_key_exists('user',$_SESSION));
+        return (array_key_exists('user', $_SESSION));
     }
 
     public function report_notLogged(): void

src/Controller.php

Spacing +8 added lines, -8 removed lines

@@ -5,14 +5,14 @@ 
 use HnrAzevedo\Validator\Validator;
 use Exception;
 
-class Controller{
+class Controller {
     use Helper;
 
     protected array $fail = [];
 
     private function checkMethod(string $method): void
     {
-        if(!method_exists($this,$method)){
+        if (!method_exists($this, $method)) {
             throw new Exception("{$method} not found in ".get_class($this).".");
         }
     }
@@ -21,7 +21,7 @@ 
     {        
         $this->ValidateData();
 
-        if($this->checkFailData()){
+        if ($this->checkFailData()) {
             return false;
         }
 
@@ -29,7 +29,7 @@ 
 
         $this->checkMethod($method);
 
-        call_user_func_array([$this,$method],  func_get_args());
+        call_user_func_array([$this, $method], func_get_args());
 
         return true;
     }
@@ -38,8 +38,8 @@ 
     {
         $valid = Validator::execute($this->getData()['POST']);
 
-        if(!$valid){
-            foreach(Validator::getErrors() as $err => $message){
+        if (!$valid) {
+            foreach (Validator::getErrors() as $err => $message) {
                 $this->fail[] = [
                     'input' => array_keys($message)[0],
                     'message' => $message[array_keys($message)[0]]
@@ -50,12 +50,12 @@ 
 
     private function checkFailData(): bool
     {
-        if(count($this->fail) > 0 ){
+        if (count($this->fail) > 0) {
             echo json_encode([
                 'error'=> $this->fail
             ]);
         }
-        return (count($this->fail) > 0 );
+        return (count($this->fail) > 0);
     }
 
 }
\ No newline at end of file

src/Helper.php

Indentation +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     }
 
     protected function ControllerForm($controller, string $method, array $values){
-		$this->checkRole();
+        $this->checkRole();
         $method = ($method !== 'method') ? $method : $this->getData()['POST']['role'];
         $data = (array_key_exists('data',$values)) ? json_decode($values['data'], true) : null;
 

Braces +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
 
         if( ($this->getProtocol() == 'form') ){
             $this->ControllerForm($controller, $method, $data['POST']);
-        }else {
+        } else {
             $data = (array_key_exists('data',$data['POST'])) ? json_decode($data['POST']['data'], true) : $data['GET'];
             call_user_func_array([$controller,$method],  $data);
         }
@@ -123,7 +123,7 @@ 
         if($route[$state] !== null){
             if(is_string($route[$state])){
                 $this->Controller($route[$state]);
-            }else{
+            } else{
                 $route[$state]();
             }
         }

Spacing +31 added lines, -31 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace HnrAzevedo\Router;
 
-trait Helper{
+trait Helper {
     use CheckTrait, ControllerTrait;
     
     private $currentRoute = null;
@@ -25,7 +25,7 @@ 
     protected function getProtocol(): string
     {
         $protocol = ((isset($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest')) ? 'ajax' : 'get';
-        $protocol = (array_key_existS('HTTP_REQUESTED_METHOD',$_SERVER)) ? strtolower($_SERVER['HTTP_REQUESTED_METHOD']) : $protocol;
+        $protocol = (array_key_existS('HTTP_REQUESTED_METHOD', $_SERVER)) ? strtolower($_SERVER['HTTP_REQUESTED_METHOD']) : $protocol;
             
         return $protocol;
     }
@@ -40,12 +40,12 @@ 
         ];
     }
 
-    protected function ControllerForm($controller, string $method, array $values){
+    protected function ControllerForm($controller, string $method, array $values) {
 		$this->checkRole();
         $method = ($method !== 'method') ? $method : $this->getData()['POST']['role'];
-        $data = (array_key_exists('data',$values)) ? json_decode($values['data'], true) : null;
+        $data = (array_key_exists('data', $values)) ? json_decode($values['data'], true) : null;
 
-        call_user_func_array([$controller,$method],  $data);
+        call_user_func_array([$controller, $method], $data);
     }
 
     protected function Controller(string $controll): void
@@ -53,7 +53,7 @@ 
         $data = $this->getData();
 
         foreach ($data['GET'] as $name => $value) {
-            $controll = str_replace('{'.$name.'}',$value,$controll);
+            $controll = str_replace('{'.$name.'}', $value, $controll);
         }
 
         $this->checkControllsettable($controll);
@@ -62,30 +62,30 @@ 
 
         $this->checkControllmethod($controll);
 
-        $controller = ucfirst(explode(':',$controll)[0]);
+        $controller = ucfirst(explode(':', $controll)[0]);
         $controller = new $controller();
-        $method = explode(':',$controll)[1];
+        $method = explode(':', $controll)[1];
 
-        if( ($this->getProtocol() == 'form') ){
+        if (($this->getProtocol() == 'form')) {
             $this->ControllerForm($controller, $method, $data['POST']);
         }else {
-            $data = (array_key_exists('data',$data['POST'])) ? json_decode($data['POST']['data'], true) : $data['GET'];
-            call_user_func_array([$controller,$method],  $data);
+            $data = (array_key_exists('data', $data['POST'])) ? json_decode($data['POST']['data'], true) : $data['GET'];
+            call_user_func_array([$controller, $method], $data);
         }
        
     }    
 
-    protected function explodeRoutes(bool $bar, string $url ,bool $bar_, string $url_): array
+    protected function explodeRoutes(bool $bar, string $url, bool $bar_, string $url_): array
     {   
-        $url = $bar ? substr($url, 0, -1) : $url ;
-        $url = explode('/',$url);
+        $url = $bar ? substr($url, 0, -1) : $url;
+        $url = explode('/', $url);
 
-        $url_ = $bar_ ? substr($url_, 0, -1) : $url_ ;
-        $url_ = explode('/',$url_);
+        $url_ = $bar_ ? substr($url_, 0, -1) : $url_;
+        $url_ = explode('/', $url_);
 
-        foreach($url as $ur => $u){
-            if(substr($u,0,2) === '{?'){
-                if(!array_key_exists($ur,$url_)){
+        foreach ($url as $ur => $u) {
+            if (substr($u, 0, 2) === '{?') {
+                if (!array_key_exists($ur, $url_)) {
                     $url_[$ur] = '';
                 };
             }
@@ -96,28 +96,28 @@ 
 
     protected function toHiking(array $route)
     {
-        $this->callOnRoute($route,'beforeAll');
-        $this->callOnRoute($route,'before');
+        $this->callOnRoute($route, 'beforeAll');
+        $this->callOnRoute($route, 'before');
 
-        if(is_string($route['role'])){
+        if (is_string($route['role'])) {
             $this->Controller($route['role']);
-            $this->callOnRoute($route,'after');
-            $this->callOnRoute($route,'afterAll');
+            $this->callOnRoute($route, 'after');
+            $this->callOnRoute($route, 'afterAll');
             return true;
         }
 
-        call_user_func_array($route['role'],$this->getData()['GET']);
+        call_user_func_array($route['role'], $this->getData()['GET']);
 
-        $this->callOnRoute($route,'after');
-        $this->callOnRoute($route,'afterAll');
+        $this->callOnRoute($route, 'after');
+        $this->callOnRoute($route, 'afterAll');
     }
 
-    protected function callOnRoute(array $route,string $state)
+    protected function callOnRoute(array $route, string $state)
     {
-        if($route[$state] !== null){
-            if(is_string($route[$state])){
+        if ($route[$state] !== null) {
+            if (is_string($route[$state])) {
                 $this->Controller($route[$state]);
-            }else{
+            }else {
                 $route[$state]();
             }
         }

src/DefinitionsTrait.php

Spacing +2 added lines, -2 removed lines

@@ -2,12 +2,12 @@
 
 namespace HnrAzevedo\Router;
 
-trait DefinitionsTrait{
+trait DefinitionsTrait {
     private static $instance = null;
 
     public static function match(string $protocols, string $uri, $walking): Router
     {
-        foreach(explode('|',$protocols) as $protocol){
+        foreach (explode('|', $protocols) as $protocol) {
             self::getInstance()->add($uri, $walking, $protocol);
         }
         return self::$instance;

src/ExtraJobsTrait.php

Spacing +9 added lines, -9 removed lines

@@ -2,11 +2,11 @@ 
 
 namespace HnrAzevedo\Router;
 
-trait ExtraJobsTrait{
+trait ExtraJobsTrait {
 
     public function before($walking): Router
     {
-        return $this->setOnRoute($walking,'before');
+        return $this->setOnRoute($walking, 'before');
     }
 
     public static function beforeAll($walking, $except = null): Router
@@ -14,12 +14,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
@@ -27,17 +27,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;
                 }
 
@@ -51,8 +51,8 @@ 
 
     private function setOnRoutes($walking, string $state, array $excepts): Router
     {
-        foreach($this->routers as $r => $route){
-            if(!in_array($this->routers,$excepts)){
+        foreach ($this->routers as $r => $route) {
+            if (!in_array($this->routers, $excepts)) {
                 $this->routers[$r][$state] = $walking;
             }
         }

src/CheckWhere.php

Spacing +19 added lines, -19 removed lines

@@ -4,18 +4,18 @@ 
 
 use Exception;
 
-trait CheckWhere{
+trait CheckWhere {
         
     protected function checkWhereParam($data)
     {
-        if(count($data) === 0){
+        if (count($data) === 0) {
             throw new Exception('It is necessary to define a condition to be tested.');
         }
     }
 
     protected function checkWhereParams($params)
     {
-        if(count($params) === 0){
+        if (count($params) === 0) {
             throw new Exception('The route in question has no parameters to be tested.');
         }
     }
@@ -24,24 +24,24 @@ 
     {
         $pass = true;
 
-        if(!is_array($route['where'])){
+        if (!is_array($route['where'])) {
             return $pass;
         }
 
-        $routeURI = explode('/',$route['url']);
+        $routeURI = explode('/', $route['url']);
         $params = [];
-        foreach($routeURI as $p => $part){
-            if(!$this->isWhered($part,$request[$p])){
+        foreach ($routeURI as $p => $part) {
+            if (!$this->isWhered($part, $request[$p])) {
                 continue;
             }
                 
-            $param = substr(str_replace('?','',$part),1,-1);
+            $param = substr(str_replace('?', '', $part), 1, -1);
 
-            if(array_key_exists($param,$route['where'])){
+            if (array_key_exists($param, $route['where'])) {
                     
                 $params[$param] = $route['where'][$param];
 
-                if(!preg_match("/^{$params[$param]}$/",$request[$p])){
+                if (!preg_match("/^{$params[$param]}$/", $request[$p])) {
                     $pass = false;
                 }
             }
@@ -52,17 +52,17 @@ 
 
     private function isWhered(string $part, string $value): bool
     {
-        return $this->isParameter($part) && !$this->checkParameterOptional($part,$value);
+        return $this->isParameter($part) && !$this->checkParameterOptional($part, $value);
     }
 
     private function checkParameterOptional(string $part, string $value): bool
     {
-        return substr($part,0,2) === '{?' && empty($value);
+        return substr($part, 0, 2) === '{?' && empty($value);
     }
 
     protected function isParameter(string $part): bool
     {
-        return (substr($part,0,1) === '{' && substr($part,-1) === '}');
+        return (substr($part, 0, 1) === '{' && substr($part, -1) === '}');
     }
 
     protected function callWhereAdd($data)
@@ -72,16 +72,16 @@ 
         $data = (count($data) > 1) ? [$data[0] => $data[1]] : $data[0];
         
         $route = end($this->routers);
-        $routeURI = explode('/',$route['url']);
+        $routeURI = explode('/', $route['url']);
         $params = [];
-        foreach($routeURI as $part){
-            if(!$this->isParameter($part)){
+        foreach ($routeURI as $part) {
+            if (!$this->isParameter($part)) {
                 continue;
             }
             
-            $param = substr(str_replace('?','',$part),1,-1);
+            $param = substr(str_replace('?', '', $part), 1, -1);
 
-            if(array_key_exists($param,$data)){
+            if (array_key_exists($param, $data)) {
                 $params[$param] = $data[$param];
             }
                 
@@ -89,7 +89,7 @@ 
 
         $this->checkWhereParams($params);
 
-        $route['where'] = (is_array($route['where'])) ? array_merge($route['where'],$params) : $params;
+        $route['where'] = (is_array($route['where'])) ? array_merge($route['where'], $params) : $params;
 
         $this->routers[count($this->routers)-1] = $route;
     }