DonUrks / phrest

src/Phrest/Middleware/HttpException.php

Spacing +3 added lines, -3 removed lines

@@ -17,11 +17,11 @@ 
     {
         try {
             $response = $delegate->process($request);
-        } catch(\Phrest\Http\Exception $e) {
+        } catch (\Phrest\Http\Exception $e) {
             $error = $e->error();
 
             $errorEntries = [];
-            foreach($error->errors() as $errorEntry) {
+            foreach ($error->errors() as $errorEntry) {
                 /** @var \Phrest\API\ErrorEntry $errorEntry */
                 $errorEntries[] = [
                     'code' => $errorEntry->code(),
@@ -44,7 +44,7 @@ 
                 $e->getCode()
             );
 
-            $this->logger->info('Exception: '.$e->getMessage(), [
+            $this->logger->info('Exception: ' . $e->getMessage(), [
                 'exception' => get_class($e),
                 'file' => $e->getFile(),
                 'line' => $e->getLine(),

src/Phrest/Middleware/Error.php

Spacing +3 added lines, -3 removed lines

@@ -15,8 +15,8 @@ 
 
     public function process(\Psr\Http\Message\ServerRequestInterface $request, \Interop\Http\ServerMiddleware\DelegateInterface $delegate)
     {
-        set_error_handler(function ($errno, $errstr, $errfile, $errline) {
-            if (! (error_reporting() & $errno)) {
+        set_error_handler(function($errno, $errstr, $errfile, $errline) {
+            if (!(error_reporting() & $errno)) {
                 return;
             }
             throw new \ErrorException($errstr, 0, $errno, $errfile, $errline);
@@ -24,7 +24,7 @@ 
 
         try {
             $response = $delegate->process($request);
-            if (! $response instanceof \Psr\Http\Message\ResponseInterface) {
+            if (!$response instanceof \Psr\Http\Message\ResponseInterface) {
                 throw new \Phrest\Exception('Application did not return a response');
             }
         } catch (\Throwable $e) {

src/Phrest/Application.php

Spacing +14 added lines, -14 removed lines

@@ -74,7 +74,7 @@ 
                     'factories' => [
                         \Zend\Expressive\Helper\UrlHelper::class => \Zend\Expressive\Helper\UrlHelperFactory::class,
 
-                        \Phrest\Application::SERVICE_LOGGER => function (\Interop\Container\ContainerInterface $container) use ($logger, $monologHandler, $monologProcessor) {
+                        \Phrest\Application::SERVICE_LOGGER => function(\Interop\Container\ContainerInterface $container) use ($logger, $monologHandler, $monologProcessor) {
                             $handlers = [];
                             foreach ($monologHandler as $handler) {
                                 $handlers[] = $container->get($handler);
@@ -86,11 +86,11 @@ 
                             return $logger;
                         },
 
-                        \Phrest\Application::SERVICE_SWAGGER => function () use ($cache, $swaggerScanDirectory) {
+                        \Phrest\Application::SERVICE_SWAGGER => function() use ($cache, $swaggerScanDirectory) {
                             return new \Phrest\Swagger($cache, $swaggerScanDirectory);
                         },
 
-                        \Phrest\Application::SERVICE_HATEOAS => function (\Interop\Container\ContainerInterface $container) use ($enableCache, $cacheDirectory) {
+                        \Phrest\Application::SERVICE_HATEOAS => function(\Interop\Container\ContainerInterface $container) use ($enableCache, $cacheDirectory) {
                             /** @var \Zend\Expressive\Router\RouterInterface $router */
                             $router = $container->get(\Zend\Expressive\Router\RouterInterface::class);
 
@@ -106,7 +106,7 @@ 
                             $hateoasBuilder->setUrlGenerator(
                                 null,
                                 new \Hateoas\UrlGenerator\CallableUrlGenerator(
-                                    function ($route, array $parameters, $absolute) use ($router, $serverUrlHelper) {
+                                    function($route, array $parameters, $absolute) use ($router, $serverUrlHelper) {
                                         $uri = $router->generateUri($route, $parameters);
                                         if ($absolute) {
                                             return $serverUrlHelper($uri);
@@ -118,11 +118,11 @@ 
                             return $hateoasBuilder->build();
                         },
 
-                        \Phrest\Application::ACTION_SWAGGER => function (\Interop\Container\ContainerInterface $container) {
+                        \Phrest\Application::ACTION_SWAGGER => function(\Interop\Container\ContainerInterface $container) {
                             return new \Phrest\API\Action\Swagger($container->get(\Phrest\Application::SERVICE_SWAGGER));
                         },
 
-                        \Phrest\Application::ACTION_ERROR_CODES => function (\Interop\Container\ContainerInterface $container) use ($cache, $errorCodes) {
+                        \Phrest\Application::ACTION_ERROR_CODES => function(\Interop\Container\ContainerInterface $container) use ($cache, $errorCodes) {
                             if ($errorCodes) {
                                 $errorCodesInstance = $container->get($errorCodes);
                             } else {
@@ -131,13 +131,13 @@ 
                             return new \Phrest\API\Action\ErrorCodes($cache, $errorCodesInstance);
                         },
 
-                        \Phrest\Application::SERVICE_HATEOAS_RESPONSE_GENERATOR => function (\Interop\Container\ContainerInterface $container) {
+                        \Phrest\Application::SERVICE_HATEOAS_RESPONSE_GENERATOR => function(\Interop\Container\ContainerInterface $container) {
                             return new \Phrest\API\HateoasResponseGenerator(
                                 $container->get(\Phrest\Application::SERVICE_HATEOAS)
                             );
                         },
 
-                        \Phrest\Application::SERVICE_REQUEST_SWAGGER_VALIDATOR => function (\Interop\Container\ContainerInterface $container) {
+                        \Phrest\Application::SERVICE_REQUEST_SWAGGER_VALIDATOR => function(\Interop\Container\ContainerInterface $container) {
                             /** @var \Phrest\Swagger $swagger */
                             $swagger = $container->get(\Phrest\Application::SERVICE_SWAGGER);
                             $jsonValidator = new \JsonSchema\Validator(
@@ -146,11 +146,11 @@ 
                             return new \Phrest\API\RequestSwaggerValidator($swagger, $jsonValidator);
                         },
 
-                        \Phrest\Application::SERVICE_ROUTER => function (\Interop\Container\ContainerInterface $container) {
+                        \Phrest\Application::SERVICE_ROUTER => function(\Interop\Container\ContainerInterface $container) {
                             return new \Zend\Expressive\Router\FastRouteRouter();
                         },
 
-                        \Zend\Expressive\Router\RouterInterface::class => function (\Interop\Container\ContainerInterface $container) {
+                        \Zend\Expressive\Router\RouterInterface::class => function(\Interop\Container\ContainerInterface $container) {
                             return $container->get(\Phrest\Application::SERVICE_ROUTER);
                         }
                     ],
@@ -158,21 +158,21 @@ 
                         \Zend\Expressive\Helper\ServerUrlHelper::class => \Zend\Expressive\Helper\ServerUrlHelper::class
                     ],
                     'initializers' => [
-                        \Psr\Log\LoggerAwareInterface::class => function (\Interop\Container\ContainerInterface $container, $service) {
+                        \Psr\Log\LoggerAwareInterface::class => function(\Interop\Container\ContainerInterface $container, $service) {
                             if ($service instanceof \Psr\Log\LoggerAwareInterface) {
                                 $service->setLogger(
                                     $container->get(\Phrest\Application::SERVICE_LOGGER)
                                 );
                             }
                         },
-                        API\RequestSwaggerValidatorAwareInterface::class => function (\Interop\Container\ContainerInterface $container, $service) {
+                        API\RequestSwaggerValidatorAwareInterface::class => function(\Interop\Container\ContainerInterface $container, $service) {
                             if ($service instanceof API\RequestSwaggerValidatorAwareInterface) {
                                 $service->setRequestSwaggerValidator(
                                     $container->get(\Phrest\Application::SERVICE_REQUEST_SWAGGER_VALIDATOR)
                                 );
                             }
                         },
-                        API\HateoasResponseGeneratorAwareInterface::class => function (\Interop\Container\ContainerInterface $container, $service) {
+                        API\HateoasResponseGeneratorAwareInterface::class => function(\Interop\Container\ContainerInterface $container, $service) {
                             if ($service instanceof API\HateoasResponseGeneratorAwareInterface) {
                                 $service->setHateoasResponseGenerator(
                                     $container->get(\Phrest\Application::SERVICE_HATEOAS_RESPONSE_GENERATOR)
@@ -206,7 +206,7 @@ 
         $app->run($request);
     }
 
-    static function createCache(bool $enableCache, ?string $cacheDirectory): \Zend\Cache\Storage\StorageInterface
+    static function createCache(bool $enableCache, ?string $cacheDirectory) : \Zend\Cache\Storage\StorageInterface
     {
         $cache = new \Zend\Cache\Storage\Adapter\BlackHole();
         if ($enableCache) {

src/Phrest/Http/StatusCodes.php

Spacing +2 added lines, -2 removed lines

@@ -18,10 +18,10 @@
     const SEE_OTHER = 303;
     const NOT_MODIFIED = 304;
     const USE_PROXY = 305;
-    const UNUSED= 306;
+    const UNUSED = 306;
     const TEMPORARY_REDIRECT = 307;
     const BAD_REQUEST = 400;
-    const UNAUTHORIZED  = 401;
+    const UNAUTHORIZED = 401;
     const PAYMENT_REQUIRED = 402;
     const FORBIDDEN = 403;
     const NOT_FOUND = 404;

src/Phrest/API/RequestSwaggerValidator.php

Spacing +3 added lines, -3 removed lines

@@ -94,7 +94,7 @@ 
         foreach ($parameters->getQueryParameters() as $parameterName => $parameter) {
             $required = $parameter['required'] ?? false;
             $value = $request->getQueryParams()[$parameterName] ?? ($parameter['default'] ?? null);
-            if(!is_null($value) || $required) {
+            if (!is_null($value) || $required) {
                 $errors = array_merge(
                     $errors,
                     $this->validateValueBySchema($value, $parameter, '{query}/' . $parameterName)
@@ -108,7 +108,7 @@ 
         foreach ($parameters->getHeaderParameters() as $parameterName => $parameter) {
             $required = $parameter['required'] ?? false;
             $value = $request->getHeader($parameterName)[0] ?? ($parameter['default'] ?? null);
-            if(!is_null($value) || $required) {
+            if (!is_null($value) || $required) {
                 $errors = array_merge(
                     $errors,
                     $this->validateValueBySchema($value, $parameter, '{header}/' . $parameterName)
@@ -145,7 +145,7 @@ 
             throw \Phrest\Http\Exception::BadRequest(
                 new \Phrest\API\Error(
                     \Phrest\API\ErrorCodes::REQUEST_PARAMETER_VALIDATION,
-                    'request parameter validation failed. OperationId: '.$operationId,
+                    'request parameter validation failed. OperationId: ' . $operationId,
                     ...$errors
                 )
             );

src/Phrest/API/Action/ErrorCodes.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
     public function process(\Psr\Http\Message\ServerRequestInterface $request, \Interop\Http\ServerMiddleware\DelegateInterface $delegate)
     {
-        if(!$this->cache->hasItem(self::CACHE_ERROR_CODES)) {
+        if (!$this->cache->hasItem(self::CACHE_ERROR_CODES)) {
             $errorCodes = $this->errorCodes->getErrorCodes();
             $this->cache->setItem(self::CACHE_ERROR_CODES, serialize($errorCodes));
         } else {

src/Phrest/API/Action/Swagger.php

Spacing +1 added lines, -1 removed lines

@@ -15,6 +15,6 @@
 
     public function process(\Psr\Http\Message\ServerRequestInterface $request, \Interop\Http\ServerMiddleware\DelegateInterface $delegate)
     {
-        return new \Zend\Diactoros\Response\TextResponse((string) $this->swagger, 200, ['Content-Type' => 'application/json']);
+        return new \Zend\Diactoros\Response\TextResponse((string)$this->swagger, 200, ['Content-Type' => 'application/json']);
     }
 }
\ No newline at end of file

src/Phrest/API/AbstractSwaggerValidatorAction.php

Spacing +2 added lines, -2 removed lines

@@ -22,9 +22,9 @@
             throw new \Phrest\Exception('no matched route found');
         }
 
-        $operationId = $method.'.'.$route->getName();
+        $operationId = $method . '.' . $route->getName();
         $operationIds = $route->getOptions()['operationIds'] ?? [];
-        if(array_key_exists($method, $operationIds)) {
+        if (array_key_exists($method, $operationIds)) {
             $operationId = $operationIds[$method];
         }