spiral / framework

src/Router/src/Traits/PipelineTrait.php

Spacing +12 added lines, -12 removed lines

@@ -46,15 +46,15 @@ 
         $route = clone $this;
 
         // array fallback
-        if (count($middleware) === 1 && is_array($middleware[0])) {
+        if (count($middleware) === 1 && is_array($middleware[0])){
             $middleware = $middleware[0];
         }
 
-        foreach ($middleware as $item) {
-            if (!is_string($item) && !$item instanceof MiddlewareInterface) {
-                if (is_object($item)) {
+        foreach ($middleware as $item){
+            if (!is_string($item) && !$item instanceof MiddlewareInterface){
+                if (is_object($item)){
                     $name = get_class($item);
-                } else {
+                }else{
                     $name = gettype($item);
                 }
 
@@ -64,7 +64,7 @@ 
             $route->middleware[] = $item;
         }
 
-        if ($route->pipeline !== null) {
+        if ($route->pipeline !== null){
             $route->pipeline = $route->makePipeline();
         }
 
@@ -81,22 +81,22 @@ 
     protected function makePipeline(): Pipeline
     {
         // pre-aggregated
-        if (count($this->middleware) === 1 && $this->middleware[0] instanceof Pipeline) {
+        if (count($this->middleware) === 1 && $this->middleware[0] instanceof Pipeline){
             return $this->middleware[0];
         }
 
-        try {
+        try{
             $pipeline = $this->container->get(Pipeline::class);
 
-            foreach ($this->middleware as $middleware) {
-                if ($middleware instanceof MiddlewareInterface) {
+            foreach ($this->middleware as $middleware){
+                if ($middleware instanceof MiddlewareInterface){
                     $pipeline->pushMiddleware($middleware);
-                } else {
+                }else{
                     // dynamically resolved
                     $pipeline->pushMiddleware($this->container->get($middleware));
                 }
             }
-        } catch (ContainerExceptionInterface $e) {
+        }catch (ContainerExceptionInterface $e){
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +27 added lines, -12 removed lines

@@ -46,15 +46,21 @@ 
         $route = clone $this;
 
         // array fallback
-        if (count($middleware) === 1 && is_array($middleware[0])) {
+        if (count($middleware) === 1 && is_array($middleware[0]))
+        {
             $middleware = $middleware[0];
         }
 
-        foreach ($middleware as $item) {
-            if (!is_string($item) && !$item instanceof MiddlewareInterface) {
-                if (is_object($item)) {
+        foreach ($middleware as $item)
+        {
+            if (!is_string($item) && !$item instanceof MiddlewareInterface)
+            {
+                if (is_object($item))
+                {
                     $name = get_class($item);
-                } else {
+                }
+                else
+                {
                     $name = gettype($item);
                 }
 
@@ -64,7 +70,8 @@ 
             $route->middleware[] = $item;
         }
 
-        if ($route->pipeline !== null) {
+        if ($route->pipeline !== null)
+        {
             $route->pipeline = $route->makePipeline();
         }
 
@@ -81,22 +88,30 @@ 
     protected function makePipeline(): Pipeline
     {
         // pre-aggregated
-        if (count($this->middleware) === 1 && $this->middleware[0] instanceof Pipeline) {
+        if (count($this->middleware) === 1 && $this->middleware[0] instanceof Pipeline)
+        {
             return $this->middleware[0];
         }
 
-        try {
+        try
+        {
             $pipeline = $this->container->get(Pipeline::class);
 
-            foreach ($this->middleware as $middleware) {
-                if ($middleware instanceof MiddlewareInterface) {
+            foreach ($this->middleware as $middleware)
+            {
+                if ($middleware instanceof MiddlewareInterface)
+                {
                     $pipeline->pushMiddleware($middleware);
-                } else {
+                }
+                else
+                {
                     // dynamically resolved
                     $pipeline->pushMiddleware($this->container->get($middleware));
                 }
             }
-        } catch (ContainerExceptionInterface $e) {
+        }
+        catch (ContainerExceptionInterface $e)
+        {
             throw new RouteException($e->getMessage(), $e->getCode(), $e);
         }
 

src/Router/src/Traits/VerbsTrait.php

Spacing +2 added lines, -2 removed lines

@@ -30,9 +30,9 @@
      */
     public function withVerbs(string ...$verbs): RouteInterface
     {
-        foreach ($verbs as &$verb) {
+        foreach ($verbs as &$verb){
             $verb = strtoupper($verb);
-            if (!in_array($verb, RouteInterface::VERBS, true)) {
+            if (!in_array($verb, RouteInterface::VERBS, true)){
                 throw new RouteException("Invalid HTTP verb `{$verb}`");
             }
 

Braces +4 added lines, -2 removed lines

@@ -30,9 +30,11 @@
      */
     public function withVerbs(string ...$verbs): RouteInterface
     {
-        foreach ($verbs as &$verb) {
+        foreach ($verbs as &$verb)
+        {
             $verb = strtoupper($verb);
-            if (!in_array($verb, RouteInterface::VERBS, true)) {
+            if (!in_array($verb, RouteInterface::VERBS, true))
+            {
                 throw new RouteException("Invalid HTTP verb `{$verb}`");
             }
 

src/Router/src/AbstractRoute.php

Spacing +2 added lines, -2 removed lines

@@ -65,12 +65,12 @@
      */
     public function match(Request $request): ?RouteInterface
     {
-        if (!in_array(strtoupper($request->getMethod()), $this->getVerbs(), true)) {
+        if (!in_array(strtoupper($request->getMethod()), $this->getVerbs(), true)){
             return null;
         }
 
         $matches = $this->uriHandler->match($request->getUri(), $this->defaults);
-        if ($matches === null) {
+        if ($matches === null){
             return null;
         }
 

Braces +4 added lines, -2 removed lines

@@ -65,12 +65,14 @@
      */
     public function match(Request $request): ?RouteInterface
     {
-        if (!in_array(strtoupper($request->getMethod()), $this->getVerbs(), true)) {
+        if (!in_array(strtoupper($request->getMethod()), $this->getVerbs(), true))
+        {
             return null;
         }
 
         $matches = $this->uriHandler->match($request->getUri(), $this->defaults);
-        if ($matches === null) {
+        if ($matches === null)
+        {
             return null;
         }
 

src/Session/tests/FactoryTest.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 {
     public function tearDown(): void
     {
-        if ((int)session_status() === PHP_SESSION_ACTIVE) {
+        if ((int)session_status() === PHP_SESSION_ACTIVE){
             session_abort();
         }
     }

Braces +2 added lines, -1 removed lines

@@ -24,7 +24,8 @@
 {
     public function tearDown(): void
     {
-        if ((int)session_status() === PHP_SESSION_ACTIVE) {
+        if ((int)session_status() === PHP_SESSION_ACTIVE)
+        {
             session_abort();
         }
     }

src/Session/tests/SessionTest.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
 
     public function tearDown(): void
     {
-        if ((int)session_status() === PHP_SESSION_ACTIVE) {
+        if ((int)session_status() === PHP_SESSION_ACTIVE){
             session_abort();
         }
     }
@@ -125,7 +125,7 @@ 
         $this->assertSame('default', $section->getName());
 
         $section->set('key', 'value');
-        foreach ($section as $key => $value) {
+        foreach ($section as $key => $value){
             $this->assertSame('key', $key);
             $this->assertSame('value', $value);
         }

Braces +4 added lines, -2 removed lines

@@ -51,7 +51,8 @@ 
 
     public function tearDown(): void
     {
-        if ((int)session_status() === PHP_SESSION_ACTIVE) {
+        if ((int)session_status() === PHP_SESSION_ACTIVE)
+        {
             session_abort();
         }
     }
@@ -125,7 +126,8 @@ 
         $this->assertSame('default', $section->getName());
 
         $section->set('key', 'value');
-        foreach ($section as $key => $value) {
+        foreach ($section as $key => $value)
+        {
             $this->assertSame('key', $key);
             $this->assertSame('value', $value);
         }

src/Session/src/SessionSection.php

Spacing +2 added lines, -2 removed lines

@@ -127,7 +127,7 @@ 
      */
     public function get(string $name, $default = null)
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name)){
             return $default;
         }
 
@@ -202,7 +202,7 @@ 
     {
         $this->session->resume();
 
-        if (!isset($_SESSION[$this->name])) {
+        if (!isset($_SESSION[$this->name])){
             $_SESSION[$this->name] = [];
         }
     }

Braces +4 added lines, -2 removed lines

@@ -127,7 +127,8 @@ 
      */
     public function get(string $name, $default = null)
     {
-        if (!$this->has($name)) {
+        if (!$this->has($name))
+        {
             return $default;
         }
 
@@ -202,7 +203,8 @@ 
     {
         $this->session->resume();
 
-        if (!isset($_SESSION[$this->name])) {
+        if (!isset($_SESSION[$this->name]))
+        {
             $_SESSION[$this->name] = [];
         }
     }

src/Session/src/Config/SessionConfig.php

Spacing +3 added lines, -3 removed lines

@@ -64,15 +64,15 @@
      */
     public function getHandler(): ?Autowire
     {
-        if (empty($this->config['handler'])) {
+        if (empty($this->config['handler'])){
             return null;
         }
 
-        if ($this->config['handler'] instanceof Autowire) {
+        if ($this->config['handler'] instanceof Autowire){
             return $this->config['handler'];
         }
 
-        if (class_exists($this->config['handler'])) {
+        if (class_exists($this->config['handler'])){
             return new Autowire($this->config['handler']);
         }
 

Braces +6 added lines, -3 removed lines

@@ -64,15 +64,18 @@
      */
     public function getHandler(): ?Autowire
     {
-        if (empty($this->config['handler'])) {
+        if (empty($this->config['handler']))
+        {
             return null;
         }
 
-        if ($this->config['handler'] instanceof Autowire) {
+        if ($this->config['handler'] instanceof Autowire)
+        {
             return $this->config['handler'];
         }
 
-        if (class_exists($this->config['handler'])) {
+        if (class_exists($this->config['handler']))
+        {
             return new Autowire($this->config['handler']);
         }
 

src/Views/tests/LoaderTest.php

Spacing +9 added lines, -9 removed lines

@@ -24,7 +24,7 @@ 
         $this->expectException(LoaderException::class);
 
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader->exists('view');
@@ -35,7 +35,7 @@ 
         $this->expectException(LoaderException::class);
 
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader->list();
@@ -46,7 +46,7 @@ 
         $this->expectException(LoaderException::class);
 
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader->load('view');
@@ -55,7 +55,7 @@ 
     public function testExists(): void
     {
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader = $loader->withExtension('php');
@@ -86,7 +86,7 @@ 
     public function testList(): void
     {
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader = $loader->withExtension('php');
@@ -103,7 +103,7 @@ 
         $this->expectException(LoaderException::class);
 
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader = $loader->withExtension('php');
@@ -146,7 +146,7 @@ 
     public function testLoad(): void
     {
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
+            'default' => __DIR__.'/fixtures/default',
         ]);
 
         $loader = $loader->withExtension('php');
@@ -169,8 +169,8 @@ 
     public function testMultipleNamespaces(): void
     {
         $loader = new ViewLoader([
-            'default' => __DIR__ . '/fixtures/default',
-            'other'   => __DIR__ . '/fixtures/other',
+            'default' => __DIR__.'/fixtures/default',
+            'other'   => __DIR__.'/fixtures/other',
 
         ]);
 

src/Views/tests/ManagerTest.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
         $manager->addEngine(new NativeEngine($this->container, 'dark.php'));
 
         $ext = [];
-        foreach ($manager->getEngines() as $e) {
+        foreach ($manager->getEngines() as $e){
             $ext[] = $e->getLoader()->getExtension();
         }
 
@@ -124,8 +124,8 @@ 
                         'directory' => '/tmp',
                     ],
                     'namespaces'   => [
-                        'default' => __DIR__ . '/fixtures/default',
-                        'other'   => __DIR__ . '/fixtures/other',
+                        'default' => __DIR__.'/fixtures/default',
+                        'other'   => __DIR__.'/fixtures/other',
                     ],
                     'dependencies' => [
 

Braces +2 added lines, -1 removed lines

@@ -36,7 +36,8 @@
         $manager->addEngine(new NativeEngine($this->container, 'dark.php'));
 
         $ext = [];
-        foreach ($manager->getEngines() as $e) {
+        foreach ($manager->getEngines() as $e)
+        {
             $ext[] = $e->getLoader()->getExtension();
         }