hnrazevedo / Router

examples/Controllers/User.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 
 namespace Example\Controllers;
 
-class User{
+class User {
 
     public function requireLogin($param, $param2): void
     {

examples/Routes/default.php

Spacing +10 added lines, -10 removed lines

@@ -4,28 +4,28 @@ 
 
 
 
-Router::beforeAll(function(){
+Router::beforeAll(function() {
     echo '<br><b>beforeAll</b><br>';
 },['testes']);
 
-Router::afterAll(function(){
+Router::afterAll(function() {
     echo '<br><b>afterAll</b><br>';
 },['testes']);
 
 
-Router::match('GET|POST|AJAX','/{1parameter}/{otherparameter}', function($parameter, $otherparameter){
+Router::match('GET|POST|AJAX', '/{1parameter}/{otherparameter}', function($parameter, $otherparameter) {
     echo "Parameter 1:{$parameter}, Parameter 2:{$otherparameter}.";
 });
 
-Router::get('/{2controller}/{method}','{controller}:{method}');
+Router::get('/{2controller}/{method}', '{controller}:{method}');
 
-Router::get('/3my-account','Controller\\User:my_account')->before(function(){
+Router::get('/3my-account', 'Controller\\User:my_account')->before(function() {
     echo '1';
 });
 
-Router::get('/4my-account/teste/teste','Controller\\User:my_account')->name('2');
+Router::get('/4my-account/teste/teste', 'Controller\\User:my_account')->name('2');
 
-Router::get('/5my-account/{:teste}/{teste2}',function($teste, $teste2){
+Router::get('/5my-account/{:teste}/{teste2}', function($teste, $teste2) {
     var_dump($teste);
     var_dump($teste2);
 })->where([
@@ -34,14 +34,14 @@ 
 ]);
 
 
-Router::get('/6my-accounttttt/{param1}/{param2}','\Example\Controllers\User@requireLogin');
+Router::get('/6my-accounttttt/{param1}/{param2}', '\Example\Controllers\User@requireLogin');
 //->middleware(['\Example\Middleware\Auth::class','Lasted']);
 
-Router::get('/5my-account/{:teste}','Controller\\User:my_account')->where([
+Router::get('/5my-account/{:teste}', 'Controller\\User:my_account')->where([
     'teste'=>'[a-zA-Z]{1,10}'
 ]);
 
 
-Router::any('/8',function(){
+Router::any('/8', function() {
     //
 })->name('index');

src/PrioritizeTrait.php

Spacing +9 added lines, -9 removed lines

@@ -2,7 +2,7 @@ 
 
 namespace HnrAzevedo\Router;
 
-trait PrioritizeTrait{
+trait PrioritizeTrait {
 
     protected array $routes = [];
 
@@ -11,27 +11,27 @@ 
         $staticRoutes = [];
         $paramRoutes = [];
 
-        foreach($this->routes as $r => $route){
+        foreach ($this->routes as $r => $route) {
 
             $path = urldecode($route['uri']->getPath());
 
-            if(strstr($path,'{')){
-                $paramRoutes[$this->getKeyArray(substr_count($path,'/') + substr_count($path,'{'),$paramRoutes)] = $route;
+            if (strstr($path, '{')) {
+                $paramRoutes[$this->getKeyArray(substr_count($path, '/')+substr_count($path, '{'), $paramRoutes)] = $route;
                 continue;    
             }
 
-            $staticRoutes[$this->getKeyArray(substr_count($path,'/'),$staticRoutes)] = $route;
+            $staticRoutes[$this->getKeyArray(substr_count($path, '/'), $staticRoutes)] = $route;
         }
 
         rsort($paramRoutes);
         rsort($staticRoutes);
 
-        $this->orderRoutes(array_merge($staticRoutes,$paramRoutes));
+        $this->orderRoutes(array_merge($staticRoutes, $paramRoutes));
     }
 
     private function getKeyArray(int $index, array $array): int
     {
-        while(array_key_exists($index,$array)){
+        while (array_key_exists($index, $array)) {
             $index++;
         }
         return $index;
@@ -40,8 +40,8 @@ 
     private function orderRoutes(array $routes):void
     {
         $kRoutes = $routes;
-        foreach($routes as $r => $route){
-            if(array_key_exists('name',$route)){
+        foreach ($routes as $r => $route) {
+            if (array_key_exists('name', $route)) {
                 unset($kRoutes[$r]);
                 $kRoutes["'{$route['name']}'"] = $route;
             }

src/WhereTrait.php

Spacing +17 added lines, -17 removed lines

@@ -11,8 +11,8 @@ 
     public static function where(array $wheres): Router
     {
         $route = self::getInstance()->inSave();
-        $route['where'] = (is_array($route['where'])) ? array_merge($route['where'],$wheres) : $wheres;
-        self::getInstance()->updateRoute($route,array_key_last(self::getInstance()->routes));
+        $route['where'] = (is_array($route['where'])) ? array_merge($route['where'], $wheres) : $wheres;
+        self::getInstance()->updateRoute($route, array_key_last(self::getInstance()->routes));
         return self::getInstance();
     }
 
@@ -22,36 +22,36 @@ 
     
         $this->parameters = [];
 
-        $uriPath .= (substr($uriPath,strlen($uriPath)-1) !== '/') ? '/' : '';
+        $uriPath .= (substr($uriPath, strlen($uriPath)-1) !== '/') ? '/' : '';
 
-        $routePath = explode('/',urldecode($route['uri']->getPath()));
+        $routePath = explode('/', urldecode($route['uri']->getPath()));
         unset($routePath[0]);
-        $uriPath = explode('/',urldecode($uriPath));
+        $uriPath = explode('/', urldecode($uriPath));
         unset($uriPath[0]);
 
         $corretRoute = true;
-        foreach($routePath as $r => $routeFrag){
+        foreach ($routePath as $r => $routeFrag) {
             $where = is_array($route['where']) ? $route['where'] : [];
             $routeFrag = $this->replaceParam($where, $routeFrag, $uriPath[$r]);
 
-            if($routeFrag !== $uriPath[$r]){
+            if ($routeFrag !== $uriPath[$r]) {
                 $corretRoute = false;
             }
         }
 
-        if(!$corretRoute){
+        if (!$corretRoute) {
             throw new \Exception('continue');
         }
 
-        $_REQUEST = array_merge($_REQUEST,$this->parameters);
+        $_REQUEST = array_merge($_REQUEST, $this->parameters);
     }
 
     private function replaceParam(array $where, string $ref, string $value): string
     {
-        if(((substr($ref,0,1) === '{') && (substr($ref,strlen($ref)-1) === '}'))) {
-            $this->parameters[str_replace(['{:','{','}'],'',$ref)] = $value;
+        if (((substr($ref, 0, 1) === '{') && (substr($ref, strlen($ref)-1) === '}'))) {
+            $this->parameters[str_replace(['{:', '{', '}'], '', $ref)] = $value;
 
-            if(array_key_exists(str_replace(['{:','{','}'],'',$ref),$where)){
+            if (array_key_exists(str_replace(['{:', '{', '}'], '', $ref), $where)) {
                 $this->matchParam($where, $ref, $value);
             }
             return $value;
@@ -61,18 +61,18 @@ 
 
     private function checkCount(string $routePath, string $uriPath): void
     {
-        $countRequest = substr_count($uriPath,'/') - substr_count($routePath,'{:');
-        $countRoute = substr_count($routePath,'/') - substr_count($routePath,'{:');
+        $countRequest = substr_count($uriPath, '/')-substr_count($routePath, '{:');
+        $countRoute = substr_count($routePath, '/')-substr_count($routePath, '{:');
 
-        if($countRequest !== $countRoute){
+        if ($countRequest !== $countRoute) {
             throw new \Exception('continue');
         }
     }
 
     private function matchParam(array $where, string $ref, string $value): void
     {
-        if(substr($ref,0,2) === '{:' || $value !== ''){
-            if(!preg_match("/^{$where[str_replace(['{:','{','}'],'',$ref)]}$/",$value)){
+        if (substr($ref, 0, 2) === '{:' || $value !== '') {
+            if (!preg_match("/^{$where[str_replace(['{:', '{', '}'], '', $ref)]}$/", $value)) {
                 throw new \Exception('continue');
             }
         }

src/RunInTrait.php

Spacing +24 added lines, -24 removed lines

@@ -13,24 +13,24 @@ 
 
     public static function before($closure): RouterInterface
     {
-        return self::addInRoute('before',$closure);
+        return self::addInRoute('before', $closure);
     }
 
     public static function after($closure): RouterInterface
     {
-        return self::addInRoute('after',$closure);
+        return self::addInRoute('after', $closure);
     }
 
     public static function beforeAll($closure, $excepts): RouterInterface
     {
-        self::getInstance()->beforeExcepts = (is_array($excepts)) ? $excepts : [ $excepts ];
+        self::getInstance()->beforeExcepts = (is_array($excepts)) ? $excepts : [$excepts];
         self::getInstance()->beforeAll = $closure;
         return self::getInstance();
     }
 
     public static function afterAll($closure, $excepts): RouterInterface
     {
-        self::getInstance()->afterExcepts = (is_array($excepts)) ? $excepts : [ $excepts ];
+        self::getInstance()->afterExcepts = (is_array($excepts)) ? $excepts : [$excepts];
         self::getInstance()->afterAll = $closure;
         return self::getInstance();
     }
@@ -47,7 +47,7 @@ 
 
     protected function executeRouteAction(): bool
     {
-        if(is_callable($this->current()['action'])){        
+        if (is_callable($this->current()['action'])) {        
             call_user_func_array($this->current()['action'], $_REQUEST);
             return true;
         }
@@ -59,12 +59,12 @@ 
     private static function addInRoutes(string $state, $closure, $excepts): RouterInterface
     {
         self::getInstance()->isInPseudGroup();
-        $excepts = (is_array($excepts)) ? $excepts : [ $excepts ];
+        $excepts = (is_array($excepts)) ? $excepts : [$excepts];
         $group = self::getInstance()->inSave()['group'];
 
-        foreach(self::getInstance()->routes as $r => $route){
-            if($route['group'] === $group && !in_array($r,$excepts)){
-                self::getInstance()->routes[$r][$state] = (is_null($route[$state])) ? [ $closure ] : array_merge($route[$state], [ $closure ]); 
+        foreach (self::getInstance()->routes as $r => $route) {
+            if ($route['group'] === $group && !in_array($r, $excepts)) {
+                self::getInstance()->routes[$r][$state] = (is_null($route[$state])) ? [$closure] : array_merge($route[$state], [$closure]); 
             }
         }
 
@@ -74,14 +74,14 @@ 
     private static function addInRoute(string $state, $closure): RouterInterface
     {
         $route = self::getInstance()->inSave();
-        $route[$state] = (!is_null($route[$state])) ? [ $closure ] : array_merge($route[$state], [ $closure ]);
-        self::updateRoute($route,array_key_last(self::getInstance()->routes));
+        $route[$state] = (!is_null($route[$state])) ? [$closure] : array_merge($route[$state], [$closure]);
+        self::updateRoute($route, array_key_last(self::getInstance()->routes));
         return self::getInstance();
     }
 
     protected function executeBefore(): void
     {
-        if(!in_array($this->currentName(),$this->beforeExcepts)){
+        if (!in_array($this->currentName(), $this->beforeExcepts)) {
             ($this->beforeAll)();
         }
 
@@ -90,7 +90,7 @@ 
 
     protected function executeAfter(): void
     {
-        if(!in_array($this->currentName(),$this->afterExcepts)){
+        if (!in_array($this->currentName(), $this->afterExcepts)) {
             ($this->afterAll)();
         }
 
@@ -99,8 +99,8 @@ 
 
     private function executeState(string $stage): void
     {
-        foreach($this->current()[$stage] as $state){
-            if(is_callable($state)){
+        foreach ($this->current()[$stage] as $state) {
+            if (is_callable($state)) {
                 $state();
                 continue;
             }
@@ -111,30 +111,30 @@ 
 
     private function executeController(string $controllerMeth): void
     {
-        if(count(explode('@',$controllerMeth)) !== 2){
+        if (count(explode('@', $controllerMeth)) !== 2) {
             throw new \RuntimeException('Misconfigured route action');
         }
 
-        $controller = (string) explode('@',$controllerMeth)[0];
-        $method = (string) explode('@',$controllerMeth)[1];
+        $controller = (string) explode('@', $controllerMeth)[0];
+        $method = (string) explode('@', $controllerMeth)[1];
 
         $this->checkControllerMeth($controllerMeth);
 
-        call_user_func_array([(new $controller()),$method], $_REQUEST);
+        call_user_func_array([(new $controller()), $method], $_REQUEST);
     }
 
     private function checkControllerMeth(string $controllerMeth): void
     {
-        $routeURI = str_replace(['{:','{','}'],'',urldecode($this->current()['uri']->getPath()));
+        $routeURI = str_replace(['{:', '{', '}'], '', urldecode($this->current()['uri']->getPath()));
 
-        $controller = (string) explode('@',$controllerMeth)[0];
-        $method = (string) explode('@',$controllerMeth)[1];
+        $controller = (string) explode('@', $controllerMeth)[0];
+        $method = (string) explode('@', $controllerMeth)[1];
 
-        if(!class_exists($controller)){
+        if (!class_exists($controller)) {
             throw new \RuntimeException("Controller not found in route URI {$routeURI}");
         }
 
-        if(!method_exists($controller, $method)){
+        if (!method_exists($controller, $method)) {
             throw new \RuntimeException("Method {$method} not found in controller {$controller} in route URI {$routeURI}");
         }
         

src/Router.php

Spacing +8 added lines, -8 removed lines

@@ -32,7 +32,7 @@ 
         $route = self::getInstance()->inSave();
         $route['name'] = $name;
         self::getInstance()->routesName[$name] = $name;
-        self::getInstance()->unsetRoute(array_key_last(self::getInstance()->routes))->updateRoute($route,$name);
+        self::getInstance()->unsetRoute(array_key_last(self::getInstance()->routes))->updateRoute($route, $name);
         return self::getInstance();
     }
 
@@ -71,15 +71,15 @@ 
         self::getInstance()->loaded = true;
         self::getInstance()->sortRoutes();
 
-        foreach(self::getInstance()->routes as $r => $route){
+        foreach (self::getInstance()->routes as $r => $route) {
             self::getInstance()->currentRoute = $route;
             self::getInstance()->currentRoute['name'] = $r;
 
-            try{
+            try {
                 self::getInstance()->checkMethod($route, $_SERVER['REQUEST_METHOD']);
                 self::getInstance()->checkData($route, (new Uri($_SERVER['REQUEST_URI']))->getPath());
                 return self::getInstance();
-            }catch(\Exception $er){
+            }catch (\Exception $er) {
                 continue;
             }
         }
@@ -90,15 +90,15 @@ 
 
     public static function run(): RouterInterface
     {
-        if(!self::getInstance()->loaded){
+        if (!self::getInstance()->loaded) {
             self::getInstance()->load();
         }
 
         self::getInstance()->executeBefore();
         
-        try{
+        try {
             self::getInstance()->executeRouteAction();
-        }catch(\Exception $er){
+        }catch (\Exception $er) {
             self::getInstance()->error = $er;
         }
         
@@ -111,7 +111,7 @@ 
 
     private function checkError(): void
     {
-        if(isset($this->error)){
+        if (isset($this->error)) {
             throw $this->error;
         }
     }

Braces +2 added lines, -2 removed lines

@@ -79,7 +79,7 @@ 
                 self::getInstance()->checkMethod($route, $_SERVER['REQUEST_METHOD']);
                 self::getInstance()->checkData($route, (new Uri($_SERVER['REQUEST_URI']))->getPath());
                 return self::getInstance();
-            }catch(\Exception $er){
+            } catch(\Exception $er){
                 continue;
             }
         }
@@ -98,7 +98,7 @@ 
         
         try{
             self::getInstance()->executeRouteAction();
-        }catch(\Exception $er){
+        } catch(\Exception $er){
             self::getInstance()->error = $er;
         }
         

src/DefinitionsTrait.php

Indentation +5 added lines, -5 removed lines

@@ -57,10 +57,10 @@ 
         
         self::checkDuplicity($uri,$method);
         
-		self::getInstance()->routes[] = [
-			'uri' => new Uri(self::getInstance()->host.self::getInstance()->prefix.$uri),
-			'action' => $closure,
-			'method' => strtoupper($method),
+        self::getInstance()->routes[] = [
+            'uri' => new Uri(self::getInstance()->host.self::getInstance()->prefix.$uri),
+            'action' => $closure,
+            'method' => strtoupper($method),
             'middlewares' => null,
             'where' => null,
             'before' => [],
@@ -75,7 +75,7 @@ 
     private static function checkDuplicity(string $uri, string $method): void
     {
         foreach(self::getInstance()->routes as $route){
-    		if( md5($route['uri'].$route['method']) === md5($uri.$method) ){
+            if( md5($route['uri'].$route['method']) === md5($uri.$method) ){
                 throw new \RuntimeException("There is already a route with the URI {$uri} and with the {$method} METHOD configured.");
             }
         }