Sam-Burns / Slim

Slim/Handlers/NotFound.php

Doc Comments +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
     /**
      * Return a response for application/json content not found
      *
-     * @return ResponseInterface
+     * @return string
      */
     protected function renderJsonNotFoundOutput()
     {
@@ -72,7 +72,7 @@ 
     /**
      * Return a response for xml content not found
      *
-     * @return ResponseInterface
+     * @return string
      */
     protected function renderXmlNotFoundOutput()
     {

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -121,6 +121,6 @@
         <a href='$homeUrl'>Visit the Home Page</a>
     </body>
 </html>
-END;
+end;
     }
 }

example/index.php

Spacing +2 added lines, -2 removed lines

@@ -26,12 +26,12 @@
  * argument for `Slim::get`, `Slim::post`, `Slim::put`, `Slim::patch`, and `Slim::delete`
  * is an anonymous function.
  */
-$app->get('/', function ($request, $response, $args) {
+$app->get('/', function($request, $response, $args) {
     $response->write("Welcome to Slim!");
     return $response;
 });
 
-$app->get('/hello[/{name}]', function ($request, $response, $args) {
+$app->get('/hello[/{name}]', function($request, $response, $args) {
     $response->write("Hello, " . $args['name']);
     return $response;
 })->setArgument('name', 'World!');

Slim/Http/Response.php

Spacing +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
      */
     protected function filterStatus($status)
     {
-        if (!is_integer($status) || $status<100 || $status>599) {
+        if (!is_integer($status) || $status < 100 || $status > 599) {
             throw new InvalidArgumentException('Invalid HTTP status code');
         }
 

Indentation +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
      *
      * @var string
      */
-     const EOL = "\r\n";
+        const EOL = "\r\n";
 
     /**
      * Create new HTTP response.

Slim/Http/Headers.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             $key = strtoupper($key);
             if (isset(static::$special[$key]) || strpos($key, 'HTTP_') === 0) {
                 if ($key !== 'HTTP_CONTENT_LENGTH') {
-                    $data[$key] =  $value;
+                    $data[$key] = $value;
                 }
             }
         }

Slim/App.php

Spacing +3 added lines, -3 removed lines

@@ -433,7 +433,7 @@ 
         $router = $this->container->get('router');
 
         // If router hasn't been dispatched or the URI changed then dispatch
-        if (null === $routeInfo || ($routeInfo['request'] !== [$request->getMethod(), (string) $request->getUri()])) {
+        if (null === $routeInfo || ($routeInfo['request'] !== [$request->getMethod(), (string)$request->getUri()])) {
             $request = $this->dispatchRouterAndPrepareRoute($request, $router);
             $routeInfo = $request->getAttribute('routeInfo');
         }
@@ -525,7 +525,7 @@ 
             $request = $request->withAttribute('route', $route);
         }
 
-        $routeInfo['request'] = [$request->getMethod(), (string) $request->getUri()];
+        $routeInfo['request'] = [$request->getMethod(), (string)$request->getUri()];
 
         return $request->withAttribute('routeInfo', $routeInfo);
     }
@@ -554,7 +554,7 @@ 
             }
             $size = $response->getBody()->getSize();
             if ($size !== null && !$response->hasHeader('Content-Length')) {
-                $response = $response->withHeader('Content-Length', (string) $size);
+                $response = $response->withHeader('Content-Length', (string)$size);
             }
         }
 

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
             }
         }
 
-        throw new \BadMethodCallException("Method $method is not a valid method");
+        throw new \BadMethodCallException("method $method is not a valid method");
     }
 
     /********************************************************************************

Unused Use Statements -2 removed lines

@@ -14,7 +14,6 @@ 
 use Throwable;
 use Closure;
 use InvalidArgumentException;
-use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\ServerRequestInterface;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Container\ContainerInterface;
@@ -26,7 +25,6 @@ 
 use Slim\Http\Headers;
 use Slim\Http\Body;
 use Slim\Http\Request;
-use Slim\Interfaces\Http\EnvironmentInterface;
 use Slim\Interfaces\RouteGroupInterface;
 use Slim\Interfaces\RouteInterface;
 use Slim\Interfaces\RouterInterface;

Slim/Handlers/AbstractError.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      */
     public function __construct($displayErrorDetails = false)
     {
-        $this->displayErrorDetails = (bool) $displayErrorDetails;
+        $this->displayErrorDetails = (bool)$displayErrorDetails;
     }
 
     /**

Slim/Container.php

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
          *
          * @return array|\ArrayAccess
          */
-        $this['settings'] = function () use ($userSettings, $defaultSettings) {
+        $this['settings'] = function() use ($userSettings, $defaultSettings) {
             return new Collection(array_merge($defaultSettings, $userSettings));
         };
 

Slim/DefaultServicesProvider.php

Spacing +10 added lines, -10 removed lines

@@ -44,7 +44,7 @@ 
              *
              * @return EnvironmentInterface
              */
-            $container['environment'] = function () {
+            $container['environment'] = function() {
                 return new Environment($_SERVER);
             };
         }
@@ -57,7 +57,7 @@ 
              *
              * @return ServerRequestInterface
              */
-            $container['request'] = function ($container) {
+            $container['request'] = function($container) {
                 return Request::createFromEnvironment($container->get('environment'));
             };
         }
@@ -70,7 +70,7 @@ 
              *
              * @return ResponseInterface
              */
-            $container['response'] = function ($container) {
+            $container['response'] = function($container) {
                 $headers = new Headers(['Content-Type' => 'text/html; charset=UTF-8']);
                 $response = new Response(200, $headers);
 
@@ -87,7 +87,7 @@ 
              *
              * @return RouterInterface
              */
-            $container['router'] = function ($container) {
+            $container['router'] = function($container) {
                 $routerCacheFile = false;
                 if (isset($container->get('settings')['routerCacheFile'])) {
                     $routerCacheFile = $container->get('settings')['routerCacheFile'];
@@ -104,7 +104,7 @@ 
              *
              * @return InvocationStrategyInterface
              */
-            $container['foundHandler'] = function () {
+            $container['foundHandler'] = function() {
                 return new RequestResponse;
             };
         }
@@ -125,7 +125,7 @@ 
              *
              * @return callable
              */
-            $container['phpErrorHandler'] = function ($container) {
+            $container['phpErrorHandler'] = function($container) {
                 return new PhpError($container->get('settings')['displayErrorDetails']);
             };
         }
@@ -146,7 +146,7 @@ 
              *
              * @return callable
              */
-            $container['errorHandler'] = function ($container) {
+            $container['errorHandler'] = function($container) {
                 return new Error($container->get('settings')['displayErrorDetails']);
             };
         }
@@ -164,7 +164,7 @@ 
              *
              * @return callable
              */
-            $container['notFoundHandler'] = function () {
+            $container['notFoundHandler'] = function() {
                 return new NotFound;
             };
         }
@@ -183,7 +183,7 @@ 
              *
              * @return callable
              */
-            $container['notAllowedHandler'] = function () {
+            $container['notAllowedHandler'] = function() {
                 return new NotAllowed;
             };
         }
@@ -196,7 +196,7 @@ 
              *
              * @return CallableResolverInterface
              */
-            $container['callableResolver'] = function ($container) {
+            $container['callableResolver'] = function($container) {
                 return new CallableResolver($container);
             };
         }

Slim/Router.php

Spacing +1 added lines, -1 removed lines

@@ -188,7 +188,7 @@
             return $this->dispatcher;
         }
 
-        $routeDefinitionCallback = function (RouteCollector $r) {
+        $routeDefinitionCallback = function(RouteCollector $r) {
             foreach ($this->getRoutes() as $route) {
                 $r->addRoute($route->getMethods(), $route->getPattern(), $route->getIdentifier());
             }