From 900694162911df5122966d2328c21c2a1b85810d Mon Sep 17 00:00:00 2001
From: Awilum <awilum@msn.com>
Date: Tue, 14 Jul 2020 21:30:42 +0300
Subject: [PATCH] refactor(core): code standard fixes #436

---
 src/flextype/Foundation/Cors/Cors.php | 52 ++++++++++++++-------------
 1 file changed, 28 insertions(+), 24 deletions(-)

diff --git a/src/flextype/Foundation/Cors/Cors.php b/src/flextype/Foundation/Cors/Cors.php
index f7073417..f09b01f8 100644
--- a/src/flextype/Foundation/Cors/Cors.php
+++ b/src/flextype/Foundation/Cors/Cors.php
@@ -9,6 +9,9 @@ declare(strict_types=1);
 
 namespace Flextype;
 
+use function count;
+use function implode;
+
 class Cors
 {
     /**
@@ -34,33 +37,34 @@ class Cors
     /**
      * Init CORS
      */
-    public function init()
+    public function init() : void
     {
         $flextype = $this->flextype;
 
-        if ($flextype['registry']->get('flextype.settings.cors.enabled')) {
-
-            $this->app->options('/{routes:.+}', function ($request, $response, $args) {
-                return $response;
-            });
-
-            $this->app->add(function ($req, $res, $next) use ($flextype) {
-                $response = $next($req, $res);
-
-                // Set variables
-                $origin  = $flextype['registry']->get('flextype.settings.cors.origin');
-                $headers = count($flextype['registry']->get('flextype.settings.cors.headers')) ? implode(', ', $flextype['registry']->get('flextype.settings.cors.headers')) : '';
-                $methods = count($flextype['registry']->get('flextype.settings.cors.methods')) ? implode(', ', $flextype['registry']->get('flextype.settings.cors.methods')) : '';
-                $expose  = count($flextype['registry']->get('flextype.settings.cors.expose')) ? implode(', ', $flextype['registry']->get('flextype.settings.cors.expose')) : '';
-                $credentials  = ($flextype['registry']->get('flextype.settings.cors.credentials')) ? true : false;
-
-                return $response
-                        ->withHeader('Access-Control-Allow-Origin', $origin)
-                        ->withHeader('Access-Control-Allow-Headers', $headers)
-                        ->withHeader('Access-Control-Allow-Methods', $methods)
-                        ->withHeader('Access-Control-Allow-Expose', $expose)
-                        ->withHeader('Access-Control-Allow-Credentials', $credentials);
-            });
+        if (! $flextype['registry']->get('flextype.settings.cors.enabled')) {
+            return;
         }
+
+        $this->app->options('/{routes:.+}', function ($request, $response, $args) {
+            return $response;
+        });
+
+        $this->app->add(function ($req, $res, $next) use ($flextype) {
+            $response = $next($req, $res);
+
+            // Set variables
+            $origin      = $flextype['registry']->get('flextype.settings.cors.origin');
+            $headers     = count($flextype['registry']->get('flextype.settings.cors.headers')) ? implode(', ', $flextype['registry']->get('flextype.settings.cors.headers')) : '';
+            $methods     = count($flextype['registry']->get('flextype.settings.cors.methods')) ? implode(', ', $flextype['registry']->get('flextype.settings.cors.methods')) : '';
+            $expose      = count($flextype['registry']->get('flextype.settings.cors.expose')) ? implode(', ', $flextype['registry']->get('flextype.settings.cors.expose')) : '';
+            $credentials = $flextype['registry']->get('flextype.settings.cors.credentials') ? true : false;
+
+            return $response
+                    ->withHeader('Access-Control-Allow-Origin', $origin)
+                    ->withHeader('Access-Control-Allow-Headers', $headers)
+                    ->withHeader('Access-Control-Allow-Methods', $methods)
+                    ->withHeader('Access-Control-Allow-Expose', $expose)
+                    ->withHeader('Access-Control-Allow-Credentials', $credentials);
+        });
     }
 }