limoncello-php-dist / core

src/Routing/Traits/HasRequestFactoryTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
     }
 
     /**
-     * @return callable
+     * @return string[]
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      */

src/Application/Application.php

Doc Comments +2 added lines, -2 removed lines

@@ -147,7 +147,7 @@ 
     }
 
     /**
-     * @return callable
+     * @return string[]
      */
     public static function getDefaultRequestFactory(): callable
     {
@@ -213,7 +213,7 @@ 
 
     /**
      * @param LimoncelloContainerInterface $container
-     * @param callable[]|null              $globalConfigurators
+     * @param callable[]              $globalConfigurators
      * @param callable[]|null              $routeConfigurators
      *
      * @return void

Spacing +9 added lines, -10 removed lines

@@ -138,8 +138,7 @@ 
             $this->addMiddlewareChain($handler, $container, $globalMiddleware, $routeMiddleware) : $handler;
 
         $request = $requestFactory === null && $hasMiddleware === false && $matchCode === RouterInterface::MATCH_FOUND ?
-            null :
-            $this->createRequest($this->sapi, $container, $requestFactory ?? static::getDefaultRequestFactory());
+            null : $this->createRequest($this->sapi, $container, $requestFactory ?? static::getDefaultRequestFactory());
 
         // Execute the pipeline by sending `Request` down all middleware (global then route's then
         // terminal handler in `Controller` and back) and then send `Response` to SAPI
@@ -206,7 +205,7 @@ 
     /**
      * @return RouterInterface|null
      */
-    protected function getRouter(): ?RouterInterface
+    protected function getRouter(): ? RouterInterface
     {
         return $this->router;
     }
@@ -278,7 +277,7 @@ 
                 ['array', PsrContainerInterface::class, ServerRequestInterface::class],
                 ResponseInterface::class
             ),
-            'Handler method should have signature ' .
+            'Handler method should have signature '.
             '`public static methodName(array, PsrContainerInterface, ServerRequestInterface): ResponseInterface`'
         );
 
@@ -326,7 +325,7 @@ 
                 [SapiInterface::class, PsrContainerInterface::class],
                 ServerRequestInterface::class
             ),
-            'Factory method should have signature ' .
+            'Factory method should have signature '.
             '`public static methodName(SapiInterface, PsrContainerInterface): ServerRequestInterface`'
         );
 
@@ -347,7 +346,7 @@ 
         array $handlerParams,
         PsrContainerInterface $container
     ): Closure {
-        return function (ServerRequestInterface $request = null) use ($handler, $handlerParams, $container) {
+        return function(ServerRequestInterface $request = null) use ($handler, $handlerParams, $container) {
             return $this->callHandler($handler, $handlerParams, $container, $request);
         };
     }
@@ -360,7 +359,7 @@ 
     private function createMethodNotAllowedTerminalHandler(array $allowedMethods): Closure
     {
         // 405 Method Not Allowed
-        return function () use ($allowedMethods) {
+        return function() use ($allowedMethods) {
             return $this->createEmptyResponse(405, ['Accept' => implode(',', $allowedMethods)]);
         };
     }
@@ -371,7 +370,7 @@ 
     private function createNotFoundTerminalHandler(): Closure
     {
         // 404 Not Found
-        return function () {
+        return function() {
             return $this->createEmptyResponse(404);
         };
     }
@@ -415,11 +414,11 @@ 
                 [ServerRequestInterface::class, Closure::class, PsrContainerInterface::class],
                 ResponseInterface::class
             ),
-            'Middleware method should have signature ' .
+            'Middleware method should have signature '.
             '`public static methodName(ServerRequestInterface, Closure, PsrContainerInterface): ResponseInterface`'
         );
 
-        return function (ServerRequestInterface $request) use ($next, $middleware, $container): ResponseInterface {
+        return function(ServerRequestInterface $request) use ($next, $middleware, $container): ResponseInterface {
             return call_user_func($middleware, $request, $next, $container);
         };
     }

src/Reflection/CheckCallableTrait.php

Spacing +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
             $methodParam = $methodParams[$index];
             if (is_string($parameter) === true) {
                 $methodType = $methodParam->getType();
-                if ($methodType === null || (string)$methodType !== $parameter) {
+                if ($methodType === null || (string) $methodType !== $parameter) {
                     $isParamOk = false;
                 }
             } elseif ($parameter === null) {
@@ -112,7 +112,7 @@ 
         $isReturnTypeOk = true;
         if ($areAllParamsOk === true && $returnType !== null) {
             $methodRetType  = $reflectionMethod->getReturnType();
-            $isReturnTypeOk = $methodRetType !== null && (string)$methodRetType === $returnType;
+            $isReturnTypeOk = $methodRetType !== null && (string) $methodRetType === $returnType;
         }
 
         $isOk = $areAllParamsOk === true && $isReturnTypeOk === true;

src/Routing/Traits/UriTrait.php

Spacing +4 added lines, -4 removed lines

@@ -30,12 +30,12 @@ 
     protected function normalizeUri(string $uri, bool $trailingSlash): string
     {
         // add starting '/' and cut ending '/' if necessary
-        $uri = strlen($uri) > 0 && $uri[0] === '/' ? $uri : '/' . $uri;
+        $uri = strlen($uri) > 0 && $uri[0] === '/' ? $uri : '/'.$uri;
         $prefixLen = strlen($uri);
         $uri = $prefixLen > 1 && substr($uri, -1) === '/' ? substr($uri, 0, $prefixLen - 1) : $uri;
 
         // feature: trailing slashes are possible when asked
-        $uri = $trailingSlash === true && substr($uri, -1) !== '/' ? $uri . '/' : $uri;
+        $uri = $trailingSlash === true && substr($uri, -1) !== '/' ? $uri.'/' : $uri;
 
         return $uri;
     }
@@ -53,12 +53,12 @@ 
 
         // only one has '/'
         if ($fEndsWithSlash xor $sStartsWithSlash) {
-            return $uri1 . $uri2;
+            return $uri1.$uri2;
         }
 
         // either both have '/' nor both don't have
 
-        $result = $fEndsWithSlash === true ? $uri1 . substr($uri2, 1) : $uri1 . '/' . $uri2;
+        $result = $fEndsWithSlash === true ? $uri1.substr($uri2, 1) : $uri1.'/'.$uri2;
 
         return $result;
     }

src/Routing/Traits/CallableTrait.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      */
     protected function getCallableToCacheMessage(): string
     {
-        return 'Value either not callable or cannot be cached or do not meet method signature requirements. ' .
+        return 'Value either not callable or cannot be cached or do not meet method signature requirements. '.
             'Use callable in form of \'ClassName::methodName\' or [ClassName::class, \'methodName\'].';
     }
 }

src/Routing/BaseGroup.php

Spacing +4 added lines, -4 removed lines

@@ -118,7 +118,7 @@ 
     /**
      * @inheritdoc
      */
-    public function parentGroup(): ?GroupInterface
+    public function parentGroup(): ? GroupInterface
     {
         return $this->parent;
     }
@@ -139,7 +139,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getName(): ?string
+    public function getName(): ? string
     {
         return $this->name;
     }
@@ -167,7 +167,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getRequestFactory(): ?callable
+    public function getRequestFactory(): ? callable
     {
         if ($this->isRequestFactorySet() === true) {
             return $this->requestFactory;
@@ -380,7 +380,7 @@ 
     /**
      * @return null|string
      */
-    private function getParentUriPrefix(): ?string
+    private function getParentUriPrefix(): ? string
     {
         $parent = $this->parentGroup();
         $result = $parent === null ? null : $parent->getUriPrefix();

src/Routing/Router.php

Spacing +4 added lines, -4 removed lines

@@ -146,11 +146,11 @@ 
     /**
      * @inheritdoc
      */
-    public function getUriPath(string $routeName): ?string
+    public function getUriPath(string $routeName): ? string
     {
         $this->checkRoutesLoaded();
 
-        list(, , $namedRouteUriPaths) = $this->cachedRoutes;
+        list(,, $namedRouteUriPaths) = $this->cachedRoutes;
 
         $result = array_key_exists($routeName, $namedRouteUriPaths) === true ? $namedRouteUriPaths[$routeName] : null;
 
@@ -168,7 +168,7 @@ 
     ): string {
         $path   = $this->getUriPath($routeName);
         $path   = $path === null ? $path : $this->replacePlaceholders($path, $placeholders);
-        $url    = empty($queryParams) === true ? "$hostUri$path" : "$hostUri$path?" . http_build_query($queryParams);
+        $url    = empty($queryParams) === true ? "$hostUri$path" : "$hostUri$path?".http_build_query($queryParams);
 
         return $url;
     }
@@ -227,7 +227,7 @@ 
                     break;
                 case '}':
                     $result .= array_key_exists($curPlaceholder, $placeholders) === true ?
-                        $placeholders[$curPlaceholder] : '{' . $curPlaceholder . '}';
+                        $placeholders[$curPlaceholder] : '{'.$curPlaceholder.'}';
                     $inPlaceholder     = false;
                     $curPlaceholder    = null;
                     $inPlaceholderName = false;

src/Routing/Route.php

Spacing +3 added lines, -3 removed lines

@@ -150,7 +150,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getRequestFactory(): ?callable
+    public function getRequestFactory(): ? callable
     {
         if ($this->isUseGroupRequestFactory() === true) {
             return $this->getGroup()->getRequestFactory();
@@ -182,9 +182,9 @@ 
     /**
      * @inheritdoc
      */
-    public function getName(): ?string
+    public function getName(): ? string
     {
-        $result = $this->name !== null ? (string)$this->getGroup()->getName() . $this->name : null;
+        $result = $this->name !== null ? (string) $this->getGroup()->getName().$this->name : null;
 
         return $result;
     }

src/Routing/Group.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     /**
      * @inheritdoc
      */
-    public function parentGroup(): ?GroupInterface
+    public function parentGroup(): ? GroupInterface
     {
         $group = parent::parentGroup();