hnrazevedo / Router

examples/Middleware/Auth.php

Spacing +2 added lines, -2 removed lines

@@ -8,11 +8,11 @@
 use Psr\Http\Server\MiddlewareInterface;
 use Psr\Http\Server\RequestHandlerInterface;
 
-class Auth implements MiddlewareInterface{
+class Auth implements MiddlewareInterface {
 
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        if(!array_key_exists('user',$_SESSION)){
+        if (!array_key_exists('user', $_SESSION)) {
             throw new Exception('Usuário não logado.');
         }
 

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::defineMiddlewares([
         'Auth'=> \Example\Middleware\Auth::class
@@ -26,7 +26,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/Routes/default.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 
 
 /* Filter example */
-Router::get('/my-account',function(){
+Router::get('/my-account', function() {
     echo 'Ok';
 })->middleware('\Example\Middleware\Auth::class');
 

src/Helper.php

Spacing +37 added lines, -37 removed lines

@@ -8,7 +8,7 @@ 
 use Exception;
 use HnrAzevedo\Http\Request;
 
-trait Helper{
+trait Helper {
     use CheckTrait, ControllerTrait;
     
     private $currentRoute = null;
@@ -35,7 +35,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;
     }
@@ -54,9 +54,9 @@ 
     {
 		$this->checkRole();
         $method = ($method !== 'method') ? $method : $this->getData()['POST']['role'];
-        $data = (array_key_exists('data',$values)) ? json_decode($values['data'], true) : [];
+        $data = (array_key_exists('data', $values)) ? json_decode($values['data'], true) : [];
 
-        call_user_func_array([$controller,$method],  $data);
+        call_user_func_array([$controller, $method], $data);
     }
 
     protected function Controller(string $controll)
@@ -64,36 +64,36 @@ 
         $data = $this->getData();
 
         foreach ($data['GET'] as $name => $value) {
-            $controll = str_replace('{'.$name.'}',$value,$controll);
+            $controll = str_replace('{'.$name.'}', $value, $controll);
         }
 
         $this->checkControllSettable($controll)->checkControllExist($controll)->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);
         }
 
         return $this;
     }    
 
-    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] = '';
                 };
             }
@@ -104,25 +104,25 @@ 
 
     protected function toHiking(array $route): bool
     {
-        $this->callOnRoute($route,'beforeAll')->callOnRoute($route,'before');
+        $this->callOnRoute($route, 'beforeAll')->callOnRoute($route, 'before');
 
-        if(is_string($route['role'])){
-            $this->Controller($route['role'])->callOnRoute($route,'after')->callOnRoute($route,'afterAll');
+        if (is_string($route['role'])) {
+            $this->Controller($route['role'])->callOnRoute($route, 'after')->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')->callOnRoute($route,'afterAll');
+        $this->callOnRoute($route, 'after')->callOnRoute($route, 'afterAll');
         return true;
     }
 
-    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]();
             }
         }
@@ -133,12 +133,12 @@ 
     {
         $currentProtocol = $this->getProtocol();
 
-        foreach(array_reverse($this->routers) as $r => $route){
+        foreach (array_reverse($this->routers) as $r => $route) {
 
             $this->currentRoute = $route;
             $this->currentRoute['name'] = $r;
 
-            if(!$this->checkProtocol($route['protocol'], $currentProtocol)){
+            if (!$this->checkProtocol($route['protocol'], $currentProtocol)) {
                 continue;
             }
 
@@ -148,23 +148,23 @@ 
             $_SERVER['REQUEST_URI'] = (array_key_exists('REQUEST_URI', $_SERVER)) ? $_SERVER['REQUEST_URI'] : '';
 
             $routs = $this->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(!$this->checkToHiking($route, $routs['routeRequest'], $routs['routeLoop'])){
+            if (!$this->checkToHiking($route, $routs['routeRequest'], $routs['routeLoop'])) {
                 continue;
             }
             
-            $this->request = new RequestHandler($route['protocol'],new Uri($this->host.$route['url']));  
-            $this->serverRequest = (new Factory())->createServerRequest($route['protocol'],new Uri($this->host.$route['url']));          
+            $this->request = new RequestHandler($route['protocol'], new Uri($this->host.$route['url']));  
+            $this->serverRequest = (new Factory())->createServerRequest($route['protocol'], new Uri($this->host.$route['url']));          
             
             $this->loaded = true;
             return $this;
         }
         
         $this->currentRoute = null;
-	    throw new Exception('Page not found.',404);
+	    throw new Exception('Page not found.', 404);
     }
 
     protected function loadByName(string $routName)
@@ -173,8 +173,8 @@ 
         $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->currentRoute = $route;

src/Router.php

Indentation +8 added lines, -8 removed lines

@@ -48,15 +48,15 @@ 
     {
         $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;
 
-    	$this->checkExistence($url,$protocol);
+        $this->checkExistence($url,$protocol);
         $this->checkTypeRole($walking);
 
-		$route = [
-			'url' => $this->prefix.$url,
-			'role' => $walking,
-			'protocol' => $protocol,
+        $route = [
+            'url' => $this->prefix.$url,
+            'role' => $walking,
+            'protocol' => $protocol,
             'middlewares' => null,
             'where' => null,
             'before' => null,
@@ -64,9 +64,9 @@ 
             'after' => null,
             'afterAll' => $this->afterAll,
             'group' => self::getInstance()->group
-		];
+        ];
 
-		$this->routers[] = $route;		
+        $this->routers[] = $route;		
         
         return self::getInstance();
     }

Spacing +16 added lines, -16 removed lines

@@ -4,7 +4,7 @@ 
 
 use HnrAzevedo\Http\ServerRequest;
 
-class Router{
+class Router {
     use Helper, DefinitionsTrait, ExtraJobsTrait;
 
     private static Router $instance;
@@ -25,7 +25,7 @@ 
 
     public static function create(): Router
     {
-        if(!self::getInstance()->instanced){
+        if (!self::getInstance()->instanced) {
             self::getInstance()->instanced = true;
         }
         
@@ -44,13 +44,13 @@ 
         return self::getInstance();
     }
 
-    public function set(string $url ,$walking , string $protocol): Router
+    public function set(string $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;
 
-    	$this->checkExistence($url,$protocol);
+    	$this->checkExistence($url, $protocol);
         $this->checkTypeRole($walking);
 
 		$route = [
@@ -73,7 +73,7 @@ 
 
     public static function group(string $prefix, callable $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;
@@ -93,11 +93,11 @@ 
 
         $currentRoute = end(self::getInstance()->routers);
 
-        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;
         }
 
@@ -117,7 +117,7 @@ 
     {
         $instance = self::create()->getInstance();
 
-        if(!$instance->loaded){
+        if (!$instance->loaded) {
             self::load($routeName);
         }
 
@@ -126,36 +126,36 @@ 
 
     public function middleware($middlewares): Router
     {
-        if($this->lastReturn !== null){
+        if ($this->lastReturn !== null) {
             $currentGroup = end($this->routers)['group'];
 
             foreach ($this->routers as $key => $value) {
-                if($value['group'] === $currentGroup){
-                    $this->routers[$key] = $this->addMiddleware($this->routers[$key],$middlewares);
+                if ($value['group'] === $currentGroup) {
+                    $this->routers[$key] = $this->addMiddleware($this->routers[$key], $middlewares);
                 }
             }
             
             return $this;
         }
         
-        $this->routers[count($this->routers)-1] = $this->addMiddleware(end($this->routers),$middlewares);
+        $this->routers[count($this->routers)-1] = $this->addMiddleware(end($this->routers), $middlewares);
         return $this;
     }
 
     public static function addMiddleware(array $route, $filter): array
     {
-        if(is_null($route['middlewares'])){
+        if (is_null($route['middlewares'])) {
             $route['middlewares'] = $filter;
             return $route;
         }
 
         $middlewares = (is_array($filter)) ? $filter : [0 => $filter];
 
-        if(is_array($route['middlewares'])){
+        if (is_array($route['middlewares'])) {
             foreach ($route['middlewares'] as $key => $value) {
                 $middlewares[] = $value;
             }
-        }else{
+        }else {
             $middlewares[] = $route['middlewares'];
         }
 

Braces +1 added lines, -1 removed lines

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

src/MiddlewareTrait.php

Spacing +8 added lines, -8 removed lines

@@ -4,15 +4,15 @@ 
 
 use Exception;
 
-trait MiddlewareTrait{
+trait MiddlewareTrait {
     protected array $middlewares = [];
 
     protected function checkMiddleware(array $route)
     {
-        $middlewares = (is_array($route['middlewares'])) ? $route['middlewares'] : [ $route['middlewares'] ];
+        $middlewares = (is_array($route['middlewares'])) ? $route['middlewares'] : [$route['middlewares']];
 
-        foreach($middlewares as $middleware){
-            if(is_null($middleware)){
+        foreach ($middlewares as $middleware) {
+            if (is_null($middleware)) {
                 continue;
             }
 
@@ -25,17 +25,17 @@ 
     protected function middlewareHandle(string $m)
     {
         $middleware = $this->middlewareExists($m);
-        $middleware->process($this->serverRequest,$this->request);
+        $middleware->process($this->serverRequest, $this->request);
     }
 
     protected function middlewareExists(string $m)
     {
-        if(class_exists(str_replace('::class','',$m))){
-            $m = str_replace('::class','',$m);
+        if (class_exists(str_replace('::class', '', $m))) {
+            $m = str_replace('::class', '', $m);
             return new $m();
         }
 
-        if(array_key_exists($m,$this->middlewares)){
+        if (array_key_exists($m, $this->middlewares)) {
             return new $this->middlewares[$m]();
         }
 

src/DefinitionsTrait.php

Spacing +2 added lines, -2 removed lines

@@ -2,12 +2,12 @@
 
 namespace HnrAzevedo\Router;
 
-trait DefinitionsTrait{
+trait DefinitionsTrait {
     private static Router $instance;
 
     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/CheckTrait.php

Spacing +18 added lines, -18 removed lines

@@ -4,7 +4,7 @@ 
 
 use Exception;
 
-trait CheckTrait{
+trait CheckTrait {
     use MiddlewareTrait, CheckWhere;
 
     protected function checkProtocol(string $expected, string $current): bool
@@ -14,7 +14,7 @@ 
 
     protected function checkName(string $routeName)
     {
-        if(!array_key_exists($routeName,$this->routers)){
+        if (!array_key_exists($routeName, $this->routers)) {
             throw new Exception('Page not found.', 404);
         }
         return $this;
@@ -22,7 +22,7 @@ 
 
     protected function checkTypeRole($role)
     {
-        if(!is_string($role) && @get_class($role) !== 'Closure' ){
+        if (!is_string($role) && @get_class($role) !== 'Closure') {
             throw new Exception("Improperly defined route track.");
         }
         return $this;
@@ -35,12 +35,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;
             }
         }
@@ -50,12 +50,12 @@ 
 
     protected function checkParameter(string $routeLoop, string $routeRequest): bool
     {
-        return !( substr($routeLoop,0,1) === '{' ) and $routeLoop !== $routeRequest;
+        return !(substr($routeLoop, 0, 1) === '{') and $routeLoop !== $routeRequest;
     }
 
     protected function checkRole()
     {
-        if(!array_key_exists('role', $_POST)){
+        if (!array_key_exists('role', $_POST)) {
             throw new Exception('O servidor não conseguiu identificar a finalidade deste formulário.');
         }
         return $this;
@@ -63,10 +63,10 @@ 
 
     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;
             }
         }
@@ -76,9 +76,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;
@@ -86,7 +86,7 @@ 
 
     protected function hasRouteName(string $name)
     {
-        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.");
         }
         return $this;
@@ -94,8 +94,8 @@ 
 
     protected function checkExistence(string $url, string $protocol)
     {
-        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.");
             }
         }
@@ -104,7 +104,7 @@ 
 
     protected function checkInGroup()
     {
-        if($this->lastReturn){
+        if ($this->lastReturn) {
             throw new Exception("At the moment it is not allowed to assign names or tests of parameters in groups..");
         }
         return $this;