platine-php / framework

src/Http/Response/FileResponse.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 

         $this->headers['content-description'] = ['File Transfer'];

         $this->headers['content-type'] = [$mimetype];

-        $this->headers['content-disposition'] = ['attachment; filename="' . $filename  . '"'];

+        $this->headers['content-disposition'] = ['attachment; filename="' . $filename . '"'];

         $this->headers['expires'] = ['0'];

         $this->headers['cache-control'] = ['must-revalidate'];

         $this->headers['pragma'] = ['public'];

Braces +4 added lines, -6 removed lines

@@ -55,25 +55,23 @@
  * @class FileResponse

  * @package Platine\Framework\Http\Response

  */

-class FileResponse extends Response

-{

+class FileResponse extends Response {
     /**

      * Create new instance

      * @param string $path

      * @param string|null $filename

      */

-    public function __construct(string $path, ?string $filename = null)

-    {

+    public function __construct(string $path, ?string $filename = null) {
         parent::__construct(200);

 

         $realpath = Path::realPath($path);

         $extension = pathinfo($realpath, PATHINFO_EXTENSION);

         $mimetype = 'application/octet-stream';

-        if (!empty($extension)) {

+        if (!empty($extension)) {
             $mimetype = Path::getMimeByExtension($extension);

         }

 

-        if (empty($filename)) {

+        if (empty($filename)) {
             $filename = basename($realpath);

         }

 

src/Http/Response/TemplateResponse.php

Braces +2 added lines, -3 removed lines

@@ -54,8 +54,7 @@ 
  * @class TemplateResponse

  * @package Platine\Framework\Http\Response

  */

-class TemplateResponse extends Response

-{

+class TemplateResponse extends Response {
     /**

      * The template instance

      * @var Template

@@ -76,7 +75,7 @@ 
         array $context = [],

         int $statusCode = 200,

         string $reasonPhrase = ''

-    ) {

+    ) {
         parent::__construct($statusCode, $reasonPhrase);

         $this->template = $template;

         $this->getBody()->write($template->render($name, $context));

src/Http/Response/JsonResponse.php

Braces +2 added lines, -3 removed lines

@@ -54,8 +54,7 @@ 
  * @class JsonResponse

  * @package Platine\Framework\Http\Response

  */

-class JsonResponse extends Response

-{

+class JsonResponse extends Response {
     /**

      * Create new instance

      * @param mixed $data

@@ -66,7 +65,7 @@ 
         $data,

         int $statusCode = 200,

         string $reasonPhrase = ''

-    ) {

+    ) {
         parent::__construct($statusCode, $reasonPhrase);

 

         $this->headers['content-type'] = ['application/json'];

src/Http/Response/RestResponse.php

Braces +4 added lines, -5 removed lines

@@ -51,8 +51,7 @@ 
  * @class RestResponse

  * @package Platine\Framework\Http\Response

  */

-class RestResponse extends JsonResponse

-{

+class RestResponse extends JsonResponse {
     /**

      * Create new instance

      * @param mixed $data

@@ -71,20 +70,20 @@ 
         string $message = '',

         int $statusCode = 200,

         string $reasonPhrase = ''

-    ) {

+    ) {
         $result = [

             'success' => $success,

             'timestamp' => time(),

             'code' => $code,

         ];

 

-        if (!empty($message)) {

+        if (!empty($message)) {
             $result['message'] = $message;

         }

 

         $result['data'] = $data;

 

-        if (!empty($extras)) {

+        if (!empty($extras)) {
             $result = array_merge($result, $extras);

         }

 

src/Http/RouteHelper.php

Braces +2 added lines, -4 removed lines

@@ -55,8 +55,7 @@ 
  * @class RouteHelper

  * @package Platine\Framework\Http

  */

-class RouteHelper

-{

+class RouteHelper {
     /**

      * The router instance

      * @var Router

@@ -67,8 +66,7 @@ 
      * Create new instance

      * @param Router $router

      */

-    public function __construct(Router $router)

-    {

+    public function __construct(Router $router) {
         $this->router = $router;

     }

 

src/Http/Middleware/RouteDispatcherMiddleware.php

Braces +4 added lines, -6 removed lines

@@ -59,8 +59,7 @@ 
  * @class RouteDispatcherMiddleware

  * @package Platine\Framework\Http\Middleware

  */

-class RouteDispatcherMiddleware implements MiddlewareInterface

-{

+class RouteDispatcherMiddleware implements MiddlewareInterface {
     /**

      * The Middleware resolver instance

      * @var MiddlewareResolverInterface

@@ -71,8 +70,7 @@ 
      * Create new instance

      * @param MiddlewareResolverInterface $resolver

      */

-    public function __construct(MiddlewareResolverInterface $resolver)

-    {

+    public function __construct(MiddlewareResolverInterface $resolver) {
         $this->resolver = $resolver;

     }

 

@@ -83,11 +81,11 @@ 
         ServerRequestInterface $request,

         RequestHandlerInterface $handler

     ): ResponseInterface {

-        if (!$result = $request->getAttribute(Route::class)) {

+        if (!$result = $request->getAttribute(Route::class)) {
             return $handler->handle($request);

         }

 

-        if ($result instanceof Route) {

+        if ($result instanceof Route) {
             $result = $result->getHandler();

         }

 

src/Http/Middleware/CorsMiddleware.php

Indentation +6 added lines, -6 removed lines

@@ -174,14 +174,14 @@
     {

         if ($this->isPreflight()) {

             $this->setOrigin()

-                 ->setMaxAge()

-                 ->setAllowCredentials()

-                 ->setAllowMethods()

-                 ->setAllowHeaders();

+                    ->setMaxAge()

+                    ->setAllowCredentials()

+                    ->setAllowMethods()

+                    ->setAllowHeaders();

         } else {

             $this->setOrigin()

-                 ->setExposedHeaders()

-                 ->setAllowCredentials();

+                    ->setExposedHeaders()

+                    ->setAllowCredentials();

         }

     }

 

Braces +18 added lines, -19 removed lines

@@ -62,8 +62,7 @@ 
  * @package Platine\Framework\Http\Middleware

  * @template T

  */

-class CorsMiddleware implements MiddlewareInterface

-{

+class CorsMiddleware implements MiddlewareInterface {
     /**

      * The configuration instance

      * @var Config<T>

@@ -96,7 +95,7 @@ 
     public function __construct(

         LoggerInterface $logger,

         Config $config

-    ) {

+    ) {
         $this->config = $config;

         $this->logger = $logger;

     }

@@ -108,13 +107,13 @@ 
         ServerRequestInterface $request,

         RequestHandlerInterface $handler

     ): ResponseInterface {

-        if (!$this->shouldBeProcessed($request)) {

+        if (!$this->shouldBeProcessed($request)) {
             return $handler->handle($request);

         }

 

         $this->request = $request;

 

-        if ($this->isPreflight()) {

+        if ($this->isPreflight()) {
             $response = new Response(204);

 

             $this->logger->info(

@@ -124,7 +123,7 @@ 
                     'url' => (string) $request->getUri(),

                 ]

             );

-        } else {

+        } else {
             $response = $handler->handle($request);

         }

         $this->response = $response;

@@ -144,13 +143,13 @@ 
         //If no route has been match no need check for CORS

         /** @var ?Route $route */

         $route = $request->getAttribute(Route::class);

-        if (!$route) {

+        if (!$route) {
             return false;

         }

 

         //Check if the path match

         $path = $this->config->get('security.cors.path', '/');

-        if (!preg_match('~^' . $path . '~', $route->getPattern())) {

+        if (!preg_match('~^' . $path . '~', $route->getPattern())) {
             return false;

         }

 

@@ -172,13 +171,13 @@ 
      */

     protected function setCorsHeaders(): void

     {

-        if ($this->isPreflight()) {

+        if ($this->isPreflight()) {
             $this->setOrigin()

                  ->setMaxAge()

                  ->setAllowCredentials()

                  ->setAllowMethods()

                  ->setAllowHeaders();

-        } else {

+        } else {
             $this->setOrigin()

                  ->setExposedHeaders()

                  ->setAllowCredentials();

@@ -195,8 +194,8 @@ 
         // default to the first allowed origin

         $origin = reset($origins);

 

-        foreach ($origins as $ori) {

-            if ($ori === $this->request->getHeaderLine('Origin')) {

+        foreach ($origins as $ori) {
+            if ($ori === $this->request->getHeaderLine('Origin')) {
                 $origin = $ori;

                 break;

             }

@@ -219,7 +218,7 @@ 
     {

 

         $headers = $this->config->get('security.cors.expose_headers', []);

-        if (!empty($headers)) {

+        if (!empty($headers)) {
             $this->response = $this->response

                                 ->withHeader(

                                     'Access-Control-Expose-Headers',

@@ -255,7 +254,7 @@ 
     {

         $allowCredentials = $this->config->get('security.cors.allow_credentials', false);

 

-        if ($allowCredentials) {

+        if ($allowCredentials) {
             $this->response = $this->response

                                 ->withHeader(

                                     'Access-Control-Allow-Credential',

@@ -274,7 +273,7 @@ 
     {

         $methods = $this->config->get('security.cors.allow_methods', []);

 

-        if (!empty($methods)) {

+        if (!empty($methods)) {
             $this->response = $this->response

                                 ->withHeader(

                                     'Access-Control-Allow-Methods',

@@ -294,17 +293,17 @@ 
 

         $headers = $this->config->get('security.cors.allow_headers', []);

 

-        if (empty($headers)) {

+        if (empty($headers)) {
             //use request headers

             $requestHeaders = $this->request

                                     ->getHeaderLine('Access-Control-Request-Headers');

 

-            if (!empty($requestHeaders)) {

+            if (!empty($requestHeaders)) {
                 $headers = $requestHeaders;

             }

         }

-        if (!empty($headers)) {

-            if (is_array($headers)) {

+        if (!empty($headers)) {
+            if (is_array($headers)) {
                 $headers = implode(', ', $headers);

             }

 

src/Http/Middleware/MaintenanceMiddleware.php

Indentation +2 added lines, -2 removed lines

@@ -138,8 +138,8 @@
             );

 

             $httpException->setTitle('Service Unavailable')

-                          ->setDescription($message)

-                          ->setHeaders($this->getHeaders($data));

+                            ->setDescription($message)

+                            ->setHeaders($this->getHeaders($data));

 

             // TODO Add headers

             throw $httpException;

Braces +15 added lines, -16 removed lines

@@ -71,8 +71,7 @@ 
  * @package Platine\Framework\Http\Middleware

  * @template T

  */

-class MaintenanceMiddleware implements MiddlewareInterface

-{

+class MaintenanceMiddleware implements MiddlewareInterface {
     /**

      * The configuration instance

      * @var Config<T>

@@ -93,7 +92,7 @@ 
     public function __construct(

         Config $config,

         Application $app

-    ) {

+    ) {
         $this->config = $config;

         $this->app = $app;

     }

@@ -106,26 +105,26 @@ 
         RequestHandlerInterface $handler

     ): ResponseInterface {

 

-        if ($this->isExceptUrl($request)) {

+        if ($this->isExceptUrl($request)) {
             return $handler->handle($request);

         }

 

-        if ($this->app->isInMaintenance()) {

-            try {

+        if ($this->app->isInMaintenance()) {
+            try {
                 $data = $this->app->maintenance()->data();

-            } catch (Throwable $ex) {

+            } catch (Throwable $ex) {
                 throw $ex;

             }

 

-            if (isset($data['secret']) && trim($request->getUri()->getPath(), '/') === $data['secret']) {

+            if (isset($data['secret']) && trim($request->getUri()->getPath(), '/') === $data['secret']) {
                 return $this->bypassResponse($data['secret']);

             }

 

-            if ($this->hasValidBypassCookie($request, $data)) {

+            if ($this->hasValidBypassCookie($request, $data)) {
                 return $handler->handle($request);

             }

 

-            if (isset($data['template'])) {

+            if (isset($data['template'])) {
                 return $this->templateResponse($data['template'], $data);

             }

 

@@ -179,7 +178,7 @@ 
             $data['status'] ?? 503

         );

 

-        foreach ($this->getHeaders($data) as $name => $value) {

+        foreach ($this->getHeaders($data) as $name => $value) {
             $response = $response->withAddedHeader($name, $value);

         }

 

@@ -194,8 +193,8 @@ 
     protected function isExceptUrl(ServerRequestInterface $request): bool

     {

         $path = $request->getUri()->getPath();

-        foreach ($this->getExceptUrls() as $url) {

-            if (Str::is($url, $path)) {

+        foreach ($this->getExceptUrls() as $url) {
+            if (Str::is($url, $path)) {
                 return true;

             }

         }

@@ -220,7 +219,7 @@ 
      */

     protected function hasValidBypassCookie(ServerRequestInterface $request, array $data): bool

     {

-        if (!isset($data['secret'])) {

+        if (!isset($data['secret'])) {
             return false;

         }

 

@@ -228,7 +227,7 @@ 
         $secret = $data['secret'];

         $name = $this->getCookieName();

         $cookieValue = (new RequestData($request))->cookie($name);

-        if (empty($cookieValue)) {

+        if (empty($cookieValue)) {
             return false;

         }

 

@@ -278,7 +277,7 @@ 
     protected function getHeaders(array $data): array

     {

         $headers = isset($data['retry']) ? ['Retry-After' => $data['retry']] : [];

-        if (isset($data['refresh'])) {

+        if (isset($data['refresh'])) {
             $headers['Refresh'] = $data['refresh'];

         }

 

src/Http/Middleware/SecurityPolicyMiddleware.php

Indentation +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
      */

     protected function shouldBeProcessed(ServerRequestInterface $request): bool

     {

-       //If no route has been match no need check for CSRF

+        //If no route has been match no need check for CSRF

         /** @var ?Route $route */

         $route = $request->getAttribute(Route::class);

         if (!$route) {

Braces +5 added lines, -7 removed lines

@@ -59,8 +59,7 @@ 
  * @package Platine\Framework\Http\Middleware

  * @template T

  */

-class SecurityPolicyMiddleware implements MiddlewareInterface

-{

+class SecurityPolicyMiddleware implements MiddlewareInterface {
     /**

      * The SecurityPolicy instance

      * @var SecurityPolicy<T>

@@ -71,8 +70,7 @@ 
      * Create new instance

      * @param SecurityPolicy<T> $securityPolicy

      */

-    public function __construct(SecurityPolicy $securityPolicy)

-    {

+    public function __construct(SecurityPolicy $securityPolicy) {
         $this->securityPolicy = $securityPolicy;

     }

 

@@ -84,7 +82,7 @@ 
         RequestHandlerInterface $handler

     ): ResponseInterface {

 

-        if (!$this->shouldBeProcessed($request)) {

+        if (!$this->shouldBeProcessed($request)) {
             return $handler->handle($request);

         }

 

@@ -102,7 +100,7 @@ 
         $response = $handler->handle($request);

 

         $headers = $this->securityPolicy->headers();

-        foreach ($headers as $name => $value) {

+        foreach ($headers as $name => $value) {
             $response = $response->withAddedHeader($name, $value);

         }

 

@@ -119,7 +117,7 @@ 
        //If no route has been match no need check for CSRF

         /** @var ?Route $route */

         $route = $request->getAttribute(Route::class);

-        if (!$route) {

+        if (!$route) {
             return false;

         }