limoncello-php-dist / core

src/Application/Application.php

Unused Use Statements -1 removed lines

@@ -28,7 +28,6 @@
 use LogicException;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
-use Zend\Diactoros\Response;
 use Zend\Diactoros\Response\EmptyResponse;
 use Zend\Diactoros\ServerRequest;
 

Spacing +4 added lines, -4 removed lines

@@ -243,7 +243,7 @@ 
         array $handlerParams,
         ContainerInterface $container
     ) {
-        return function (ServerRequestInterface $request = null) use ($handler, $handlerParams, $container) {
+        return function(ServerRequestInterface $request = null) use ($handler, $handlerParams, $container) {
             return call_user_func($handler, $handlerParams, $container, $request);
         };
     }
@@ -256,7 +256,7 @@ 
     private function createMethodNotAllowedTerminalHandler(array $allowedMethods)
     {
         // 405 Method Not Allowed
-        return function () use ($allowedMethods) {
+        return function() use ($allowedMethods) {
             return $this->createEmptyResponse(405, ['Accept' => implode(',', $allowedMethods)]);
         };
     }
@@ -267,7 +267,7 @@ 
     private function createNotFoundTerminalHandler()
     {
         // 404 Not Found
-        return function () {
+        return function() {
             return $this->createEmptyResponse(404);
         };
     }
@@ -320,7 +320,7 @@ 
      */
     private function createMiddlewareChainLink(Closure $next, callable $middleware, ContainerInterface $userContainer)
     {
-        return function (ServerRequestInterface $request) use ($next, $middleware, $userContainer) {
+        return function(ServerRequestInterface $request) use ($next, $middleware, $userContainer) {
             return call_user_func($middleware, $request, $next, $userContainer);
         };
     }

Doc Comments +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
     }
 
     /**
-     * @return callable
+     * @return string[]
      */
     public static function getDefaultRequestFactory()
     {
@@ -298,7 +298,7 @@ 
     /**
      * @param Closure            $handler
      * @param ContainerInterface $userContainer
-     * @param array|null         $globalMiddleware
+     * @param callable[]         $globalMiddleware
      * @param array|null         $routeMiddleware
      *
      * @return Closure
@@ -353,7 +353,7 @@ 
      * @param int|string $key
      * @param mixed      $default
      *
-     * @return mixed
+     * @return string
      */
     private function getValue(array $config, $key, $default)
     {

src/Routing/Group.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 class Group extends BaseGroup
 {
     /**
-     * @return null
+     * @return null|\Limoncello\Core\Contracts\Routing\GroupInterface
      */
     public function parentGroup()
     {

src/Application/Sapi.php

Spacing +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
         $this->sapiEmitter = $sapiEmitter;
 
         // returns value if not null or $fallback otherwise
-        $get = function ($nullable, $fallback) {
+        $get = function($nullable, $fallback) {
             return $nullable !== null ? $nullable : $fallback;
         };
 

src/Routing/Traits/UriTrait.php

Spacing +4 added lines, -4 removed lines

@@ -30,12 +30,12 @@ 
     protected function normalizeUri($uri, $trailingSlash)
     {
         // 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

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

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/Route.php

Spacing +1 added lines, -1 removed lines

@@ -188,7 +188,7 @@
      */
     public function getName()
     {
-        $result = $this->name !== null ? $this->getGroup()->getName() . $this->name : null;
+        $result = $this->name !== null ? $this->getGroup()->getName().$this->name : null;
 
         return $result;
     }

src/Routing/Router.php

Spacing +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
     {
         $this->checkRoutesLoaded();
 
-        list(, , $namedRouteUriPaths) = $this->cachedRoutes;
+        list(,, $namedRouteUriPaths) = $this->cachedRoutes;
 
         $result = array_key_exists($routeName, $namedRouteUriPaths) === true ? $namedRouteUriPaths[$routeName] : null;
 

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;
     }