selamiphp / router

src/Router.php

Doc Comments +4 added lines

@@ -161,6 +161,7 @@ 
      * @param  string                      $action
      * @param  int                         $returnType
      * @param  string                      $alias
+     * @param string $requestMethods
      * @throws InvalidArgumentException
      * @throws UnexpectedValueException
      */
@@ -268,6 +269,9 @@ 
         return new FastRoute\Dispatcher\GroupCountBased($routeCollector->getData());
     }
 
+    /**
+     * @param FastRoute\RouteCollector $routeCollector
+     */
     private function createCachedRoute($routeCollector) : void
     {
         if ($this->cachedFile !== null && !file_exists($this->cachedFile)) {

Spacing +12 added lines, -12 removed lines

@@ -123,7 +123,7 @@ 
         string $method,
         string $requestedPath,
         string $folder = '',
-        ?string $cachedFile = null
+        ? string $cachedFile = null
     ) {
         if (!in_array($method, self::$validRequestMethods, true)) {
             $message = sprintf('%s is not valid Http request method.', $method);
@@ -131,7 +131,7 @@ 
         }
         $this->method = $method;
         $this->requestedPath = $this->extractFolder($requestedPath, $folder);
-        $this->defaultReturnType = ($defaultReturnType >=1 && $defaultReturnType <=7) ? $defaultReturnType : self::HTML;
+        $this->defaultReturnType = ($defaultReturnType >= 1 && $defaultReturnType <= 7) ? $defaultReturnType : self::HTML;
         $this->cachedFile = $cachedFile;
     }
 
@@ -168,8 +168,8 @@ 
         $requestMethods,
         string $route,
         string $action,
-        ?int $returnType = null,
-        ?string $alias = null
+        ? int $returnType = null,
+        ? string $alias = null
     ) : void {
     
         $requestMethodsGiven = is_array($requestMethods) ? (array) $requestMethods : [0 => $requestMethods];
@@ -207,12 +207,12 @@ 
      * @param int|null $returnType
      * @return int
      */
-    private function determineReturnType(?int $returnType) : int
+    private function determineReturnType(? int $returnType) : int
     {
         if ($returnType === null) {
             return $this->defaultReturnType;
         }
-        return ($returnType >=1 && $returnType <=7) ? $returnType : self::HTML;
+        return ($returnType >= 1 && $returnType <= 7) ? $returnType : self::HTML;
     }
 
     /**
@@ -299,23 +299,23 @@ 
      */
     private function addRoutes(FastRoute\RouteCollector $route) : void
     {
-        $routeIndex=0;
+        $routeIndex = 0;
         foreach ($this->routes as $definedRoute) {
             $definedRoute[3] = $definedRoute[3] ?? $this->defaultReturnType;
-            $routeName = 'routeClosure'.$routeIndex;
+            $routeName = 'routeClosure' . $routeIndex;
             $route->addRoute(strtoupper($definedRoute[0]), $definedRoute[1], $routeName);
             $routeIndex++;
         }
     }
     private function setRouteClosures() : void
     {
-        $routeIndex=0;
+        $routeIndex = 0;
         foreach ($this->routes as $definedRoute) {
             $definedRoute[3] = $definedRoute[3] ?? $this->defaultReturnType;
-            $routeName = 'routeClosure'.$routeIndex;
+            $routeName = 'routeClosure' . $routeIndex;
             [$requestMedhod, $url, $controller, $returnType] = $definedRoute;
-            $returnType = ($returnType >=1 && $returnType <=7) ? $returnType : $this->defaultReturnType;
-            $this->routerClosures[$routeName]= function ($args) use ($controller, $returnType) {
+            $returnType = ($returnType >= 1 && $returnType <= 7) ? $returnType : $this->defaultReturnType;
+            $this->routerClosures[$routeName] = function($args) use ($controller, $returnType) {
                 return  ['controller' => $controller, 'returnType'=> $returnType, 'args'=> $args];
             };
             $routeIndex++;