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/Routing/NestedGroup.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
     {
         $parentGroupName = $this->parentGroup()->getName();
         $selfName        = parent::getName();
-        $result          = $parentGroupName !== null || $selfName !== null ? $parentGroupName . $selfName : null;
+        $result          = $parentGroupName !== null || $selfName !== null ? $parentGroupName.$selfName : null;
 
         return $result;
     }

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, -9 removed lines

@@ -200,7 +200,7 @@ 
     protected function handleThrowable(
         Throwable $throwable,
         ?PsrContainerInterface $container
-    ): ThrowableResponseInterface {
+    ) : ThrowableResponseInterface {
         if ($container !== null && $container->has(ThrowableHandlerInterface::class) === true) {
             /** @var ThrowableHandlerInterface $handler */
             $handler  = $container->get(ThrowableHandlerInterface::class);
@@ -243,7 +243,7 @@ 
              */
             public function __construct(Throwable $throwable, int $status)
             {
-                parent::__construct((string)$throwable, $status);
+                parent::__construct((string) $throwable, $status);
                 $this->setThrowable($throwable);
             }
         };
@@ -326,7 +326,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`'
         );
 
@@ -374,7 +374,7 @@ 
                 [SapiInterface::class, PsrContainerInterface::class],
                 ServerRequestInterface::class
             ),
-            'Factory method should have signature ' .
+            'Factory method should have signature '.
             '`public static methodName(SapiInterface, PsrContainerInterface): ServerRequestInterface`'
         );
 
@@ -395,7 +395,7 @@ 
         array $handlerParams,
         PsrContainerInterface $container
     ): Closure {
-        return function (ServerRequestInterface $request = null) use (
+        return function(ServerRequestInterface $request = null) use (
             $handler,
             $handlerParams,
             $container
@@ -416,7 +416,7 @@ 
     private function createMethodNotAllowedTerminalHandler(array $allowedMethods): Closure
     {
         // 405 Method Not Allowed
-        return function () use ($allowedMethods): ResponseInterface {
+        return function() use ($allowedMethods): ResponseInterface {
             return $this->createEmptyResponse(405, ['Accept' => implode(',', $allowedMethods)]);
         };
     }
@@ -427,7 +427,7 @@ 
     private function createNotFoundTerminalHandler(): Closure
     {
         // 404 Not Found
-        return function (): ResponseInterface {
+        return function(): ResponseInterface {
             return $this->createEmptyResponse(404);
         };
     }
@@ -471,11 +471,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/Route.php

Spacing +2 added lines, -2 removed lines

@@ -150,7 +150,7 @@ 
     /**
      * @inheritdoc
      */
-    public function getRequestFactory(): ?callable
+    public function getRequestFactory(): ? callable
     {
         if ($this->isUseGroupRequestFactory() === true) {
             return $this->getGroup()->getRequestFactory();
@@ -184,7 +184,7 @@ 
      */
     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/BaseGroup.php

Spacing +1 added lines, -1 removed lines

@@ -167,7 +167,7 @@
     /**
      * @inheritdoc
      */
-    public function getRequestFactory(): ?callable
+    public function getRequestFactory(): ? callable
     {
         if ($this->isRequestFactorySet() === true) {
             return $this->requestFactory;

src/Routing/Router.php

Spacing +3 added lines, -3 removed lines

@@ -150,7 +150,7 @@ 
     {
         $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;