Passed
Pull Request — master (#927)
by Aleksei
08:21
created
src/Router/tests/Diactoros/ResponseFactory.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@
 block discarded – undo
33 33
         $response = new Response($code);
34 34
         $response = $response->withStatus($code, $reasonPhrase);
35 35
 
36
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
36
+        foreach ($this->config->getBaseHeaders() as $header => $value){
37 37
             $response = $response->withAddedHeader($header, $value);
38 38
         }
39 39
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,8 @@
 block discarded – undo
33 33
         $response = new Response($code);
34 34
         $response = $response->withStatus($code, $reasonPhrase);
35 35
 
36
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
36
+        foreach ($this->config->getBaseHeaders() as $header => $value)
37
+        {
37 38
             $response = $response->withAddedHeader($header, $value);
38 39
         }
39 40
 
Please login to merge, or discard this patch.
src/Router/src/Loader/Configurator/RoutingConfigurator.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,19 +14,19 @@
 block discarded – undo
14 14
     public function __construct(
15 15
         private readonly RouteCollection $collection,
16 16
         private readonly LoaderInterface $loader
17
-    ) {
17
+    ){
18 18
     }
19 19
 
20
-    public function import(string|array $resource, string $type = null): ImportConfigurator
20
+    public function import(string | array $resource, string $type = null): ImportConfigurator
21 21
     {
22 22
         $imported = $this->loader->load($resource, $type) ?: [];
23 23
 
24
-        if (!\is_array($imported)) {
24
+        if (!\is_array($imported)){
25 25
             return new ImportConfigurator($this->collection, $imported);
26 26
         }
27 27
 
28 28
         $mergedCollection = new RouteCollection();
29
-        foreach ($imported as $subCollection) {
29
+        foreach ($imported as $subCollection){
30 30
             $mergedCollection->addCollection($subCollection);
31 31
         }
32 32
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,12 +21,14 @@
 block discarded – undo
21 21
     {
22 22
         $imported = $this->loader->load($resource, $type) ?: [];
23 23
 
24
-        if (!\is_array($imported)) {
24
+        if (!\is_array($imported))
25
+        {
25 26
             return new ImportConfigurator($this->collection, $imported);
26 27
         }
27 28
 
28 29
         $mergedCollection = new RouteCollection();
29
-        foreach ($imported as $subCollection) {
30
+        foreach ($imported as $subCollection)
31
+        {
30 32
             $mergedCollection->addCollection($subCollection);
31 33
         }
32 34
 
Please login to merge, or discard this patch.
src/Router/src/Loader/Configurator/RouteConfigurator.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -31,12 +31,12 @@  discard block
 block discarded – undo
31 31
         private readonly string $name,
32 32
         private readonly string $pattern,
33 33
         private readonly RouteCollection $collection
34
-    ) {
34
+    ){
35 35
     }
36 36
 
37 37
     public function __destruct()
38 38
     {
39
-        if ($this->target === null) {
39
+        if ($this->target === null){
40 40
             throw new TargetException(
41 41
                 \sprintf('The [%s] route has no defined target. Call one of: `controller`, `action`, 
42 42
                     `namespaced`, `groupControllers`, `callable`, `handler` methods.', $this->name)
@@ -87,21 +87,21 @@  discard block
 block discarded – undo
87 87
         return $this;
88 88
     }
89 89
 
90
-    public function action(string $controller, string|array $action, int $options = 0): self
90
+    public function action(string $controller, string | array $action, int $options = 0): self
91 91
     {
92 92
         $this->target = new Action($controller, $action, $options);
93 93
 
94 94
         return $this;
95 95
     }
96 96
 
97
-    public function callable(array|\Closure $callable): self
97
+    public function callable(array | \Closure $callable): self
98 98
     {
99 99
         $this->target = $callable;
100 100
 
101 101
         return $this;
102 102
     }
103 103
 
104
-    public function handler(TargetInterface|string $target): self
104
+    public function handler(TargetInterface | string $target): self
105 105
     {
106 106
         $this->target = $target;
107 107
 
@@ -136,9 +136,9 @@  discard block
 block discarded – undo
136 136
         return $this;
137 137
     }
138 138
 
139
-    public function middleware(MiddlewareInterface|string|array $middleware): self
139
+    public function middleware(MiddlewareInterface | string | array $middleware): self
140 140
     {
141
-        if (!\is_array($middleware)) {
141
+        if (!\is_array($middleware)){
142 142
             $middleware = [$middleware];
143 143
         }
144 144
 
@@ -147,9 +147,9 @@  discard block
 block discarded – undo
147 147
         return $this;
148 148
     }
149 149
 
150
-    public function methods(string|array $methods): self
150
+    public function methods(string | array $methods): self
151 151
     {
152
-        $this->methods = (array) $methods;
152
+        $this->methods = (array)$methods;
153 153
 
154 154
         return $this;
155 155
     }
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -36,7 +36,8 @@  discard block
 block discarded – undo
36 36
 
37 37
     public function __destruct()
38 38
     {
39
-        if ($this->target === null) {
39
+        if ($this->target === null)
40
+        {
40 41
             throw new TargetException(
41 42
                 \sprintf('The [%s] route has no defined target. Call one of: `controller`, `action`, 
42 43
                     `namespaced`, `groupControllers`, `callable`, `handler` methods.', $this->name)
@@ -138,7 +139,8 @@  discard block
 block discarded – undo
138 139
 
139 140
     public function middleware(MiddlewareInterface|string|array $middleware): self
140 141
     {
141
-        if (!\is_array($middleware)) {
142
+        if (!\is_array($middleware))
143
+        {
142 144
             $middleware = [$middleware];
143 145
         }
144 146
 
Please login to merge, or discard this patch.
src/Router/src/Loader/DelegatingLoader.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -10,12 +10,12 @@
 block discarded – undo
10 10
 {
11 11
     public function __construct(
12 12
         private readonly LoaderRegistryInterface $registry
13
-    ) {
13
+    ){
14 14
     }
15 15
 
16 16
     public function load(mixed $resource, string $type = null): mixed
17 17
     {
18
-        if (false === $loader = $this->registry->resolve($resource, $type)) {
18
+        if (false === $loader = $this->registry->resolve($resource, $type)){
19 19
             throw new LoaderLoadException(\sprintf('Loader for type [%s] not found.', $type));
20 20
         }
21 21
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,8 @@
 block discarded – undo
15 15
 
16 16
     public function load(mixed $resource, string $type = null): mixed
17 17
     {
18
-        if (false === $loader = $this->registry->resolve($resource, $type)) {
18
+        if (false === $loader = $this->registry->resolve($resource, $type))
19
+        {
19 20
             throw new LoaderLoadException(\sprintf('Loader for type [%s] not found.', $type));
20 21
         }
21 22
 
Please login to merge, or discard this patch.
src/Router/src/Loader/LoaderRegistryInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,5 +11,5 @@
 block discarded – undo
11 11
      *
12 12
      * @param string|null $type The resource type or null if unknown
13 13
      */
14
-    public function resolve(mixed $resource, string $type = null): LoaderInterface|false;
14
+    public function resolve(mixed $resource, string $type = null): LoaderInterface | false;
15 15
 }
Please login to merge, or discard this patch.
src/Router/src/Loader/LoaderRegistry.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -16,15 +16,15 @@
 block discarded – undo
16 16
      */
17 17
     public function __construct(array $loaders = [])
18 18
     {
19
-        foreach ($loaders as $loader) {
19
+        foreach ($loaders as $loader){
20 20
             $this->addLoader($loader);
21 21
         }
22 22
     }
23 23
 
24
-    public function resolve(mixed $resource, string $type = null): LoaderInterface|false
24
+    public function resolve(mixed $resource, string $type = null): LoaderInterface | false
25 25
     {
26
-        foreach ($this->loaders as $loader) {
27
-            if ($loader->supports($resource, $type)) {
26
+        foreach ($this->loaders as $loader){
27
+            if ($loader->supports($resource, $type)){
28 28
                 return $loader;
29 29
             }
30 30
         }
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,15 +16,18 @@
 block discarded – undo
16 16
      */
17 17
     public function __construct(array $loaders = [])
18 18
     {
19
-        foreach ($loaders as $loader) {
19
+        foreach ($loaders as $loader)
20
+        {
20 21
             $this->addLoader($loader);
21 22
         }
22 23
     }
23 24
 
24 25
     public function resolve(mixed $resource, string $type = null): LoaderInterface|false
25 26
     {
26
-        foreach ($this->loaders as $loader) {
27
-            if ($loader->supports($resource, $type)) {
27
+        foreach ($this->loaders as $loader)
28
+        {
29
+            if ($loader->supports($resource, $type))
30
+            {
28 31
                 return $loader;
29 32
             }
30 33
         }
Please login to merge, or discard this patch.
src/Router/src/Loader/PhpFileLoader.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
     public function __construct(
17 17
         private readonly FactoryInterface $factory,
18 18
         private readonly ResolverInterface $resolver,
19
-    ) {
19
+    ){
20 20
     }
21 21
 
22 22
     /**
@@ -24,11 +24,11 @@  discard block
 block discarded – undo
24 24
      */
25 25
     public function load(mixed $resource, string $type = null): RouteCollection
26 26
     {
27
-        if (!\file_exists($resource)) {
27
+        if (!\file_exists($resource)){
28 28
             throw new LoaderLoadException(\sprintf('File [%s] does not exist.', $resource));
29 29
         }
30 30
 
31
-        $load = static function (string $path) {
31
+        $load = static function (string $path){
32 32
             return include $path;
33 33
         };
34 34
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,11 +24,13 @@
 block discarded – undo
24 24
      */
25 25
     public function load(mixed $resource, string $type = null): RouteCollection
26 26
     {
27
-        if (!\file_exists($resource)) {
27
+        if (!\file_exists($resource))
28
+        {
28 29
             throw new LoaderLoadException(\sprintf('File [%s] does not exist.', $resource));
29 30
         }
30 31
 
31
-        $load = static function (string $path) {
32
+        $load = static function (string $path)
33
+        {
32 34
             return include $path;
33 35
         };
34 36
 
Please login to merge, or discard this patch.
src/Router/src/PipelineFactory.php 2 patches
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
     public function __construct(
18 18
         private readonly ContainerInterface $container,
19 19
         private readonly FactoryInterface $factory
20
-    ) {
20
+    ){
21 21
     }
22 22
 
23 23
     /**
@@ -26,27 +26,27 @@  discard block
 block discarded – undo
26 26
      */
27 27
     public function createWithMiddleware(array $middleware): Pipeline
28 28
     {
29
-        if (\count($middleware) === 1 && $middleware[0] instanceof Pipeline) {
29
+        if (\count($middleware) === 1 && $middleware[0] instanceof Pipeline){
30 30
             return $middleware[0];
31 31
         }
32 32
 
33 33
         $pipeline = $this->factory->make(Pipeline::class);
34 34
         \assert($pipeline instanceof Pipeline);
35 35
 
36
-        foreach ($middleware as $item) {
37
-            if ($item instanceof MiddlewareInterface) {
36
+        foreach ($middleware as $item){
37
+            if ($item instanceof MiddlewareInterface){
38 38
                 $pipeline->pushMiddleware($item);
39
-            } elseif (\is_string($item)) {
39
+            } elseif (\is_string($item)){
40 40
                 $item = $this->container->get($item);
41 41
                 \assert($item instanceof MiddlewareInterface);
42 42
 
43 43
                 $pipeline->pushMiddleware($item);
44
-            } elseif ($item instanceof Autowire) {
44
+            } elseif ($item instanceof Autowire){
45 45
                 $item = $item->resolve($this->factory);
46 46
                 \assert($item instanceof MiddlewareInterface);
47 47
 
48 48
                 $pipeline->pushMiddleware($item);
49
-            } else {
49
+            }else{
50 50
                 $name = get_debug_type($item);
51 51
                 throw new RouteException(\sprintf('Invalid middleware `%s`', $name));
52 52
             }
Please login to merge, or discard this patch.
Braces   +15 added lines, -6 removed lines patch added patch discarded remove patch
@@ -26,27 +26,36 @@
 block discarded – undo
26 26
      */
27 27
     public function createWithMiddleware(array $middleware): Pipeline
28 28
     {
29
-        if (\count($middleware) === 1 && $middleware[0] instanceof Pipeline) {
29
+        if (\count($middleware) === 1 && $middleware[0] instanceof Pipeline)
30
+        {
30 31
             return $middleware[0];
31 32
         }
32 33
 
33 34
         $pipeline = $this->factory->make(Pipeline::class);
34 35
         \assert($pipeline instanceof Pipeline);
35 36
 
36
-        foreach ($middleware as $item) {
37
-            if ($item instanceof MiddlewareInterface) {
37
+        foreach ($middleware as $item)
38
+        {
39
+            if ($item instanceof MiddlewareInterface)
40
+            {
38 41
                 $pipeline->pushMiddleware($item);
39
-            } elseif (\is_string($item)) {
42
+            }
43
+            elseif (\is_string($item))
44
+            {
40 45
                 $item = $this->container->get($item);
41 46
                 \assert($item instanceof MiddlewareInterface);
42 47
 
43 48
                 $pipeline->pushMiddleware($item);
44
-            } elseif ($item instanceof Autowire) {
49
+            }
50
+            elseif ($item instanceof Autowire)
51
+            {
45 52
                 $item = $item->resolve($this->factory);
46 53
                 \assert($item instanceof MiddlewareInterface);
47 54
 
48 55
                 $pipeline->pushMiddleware($item);
49
-            } else {
56
+            }
57
+            else
58
+            {
50 59
                 $name = get_debug_type($item);
51 60
                 throw new RouteException(\sprintf('Invalid middleware `%s`', $name));
52 61
             }
Please login to merge, or discard this patch.
src/Router/src/AbstractRoute.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
     public function __construct(
21 21
         protected string $pattern,
22 22
         array $defaults = []
23
-    ) {
23
+    ){
24 24
         $this->defaults = $defaults;
25 25
     }
26 26
 
@@ -39,12 +39,12 @@  discard block
 block discarded – undo
39 39
 
40 40
     public function match(Request $request): ?static
41 41
     {
42
-        if (!\in_array(\strtoupper($request->getMethod()), $this->getVerbs(), true)) {
42
+        if (!\in_array(\strtoupper($request->getMethod()), $this->getVerbs(), true)){
43 43
             return null;
44 44
         }
45 45
 
46 46
         $matches = $this->uriHandler->match($request->getUri(), $this->defaults);
47
-        if ($matches === null) {
47
+        if ($matches === null){
48 48
             return null;
49 49
         }
50 50
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,12 +39,14 @@
 block discarded – undo
39 39
 
40 40
     public function match(Request $request): ?static
41 41
     {
42
-        if (!\in_array(\strtoupper($request->getMethod()), $this->getVerbs(), true)) {
42
+        if (!\in_array(\strtoupper($request->getMethod()), $this->getVerbs(), true))
43
+        {
43 44
             return null;
44 45
         }
45 46
 
46 47
         $matches = $this->uriHandler->match($request->getUri(), $this->defaults);
47
-        if ($matches === null) {
48
+        if ($matches === null)
49
+        {
48 50
             return null;
49 51
         }
50 52
 
Please login to merge, or discard this patch.