Passed
Pull Request — master (#1115)
by Maxim
22:29
created
src/Config/src/Patch/Append.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -19,21 +19,21 @@
 block discarded – undo
19 19
         string $position,
20 20
         private ?string $key,
21 21
         private mixed $value
22
-    ) {
22
+    ){
23 23
         $this->position = $position === '.' ? '' : $position;
24 24
     }
25 25
 
26 26
     public function patch(array $config): array
27 27
     {
28
-        try {
28
+        try{
29 29
             $target = &$this->dotGet($config, $this->position);
30 30
 
31
-            if ($this->key !== null) {
31
+            if ($this->key !== null){
32 32
                 $target[$this->key] = $this->value;
33
-            } else {
33
+            }else{
34 34
                 $target[] = $this->value;
35 35
             }
36
-        } catch (DotNotFoundException $e) {
36
+        }catch (DotNotFoundException $e){
37 37
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
38 38
         }
39 39
 
Please login to merge, or discard this patch.
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -25,15 +25,21 @@
 block discarded – undo
25 25
 
26 26
     public function patch(array $config): array
27 27
     {
28
-        try {
28
+        try
29
+        {
29 30
             $target = &$this->dotGet($config, $this->position);
30 31
 
31
-            if ($this->key !== null) {
32
+            if ($this->key !== null)
33
+            {
32 34
                 $target[$this->key] = $this->value;
33
-            } else {
35
+            }
36
+            else
37
+            {
34 38
                 $target[] = $this->value;
35 39
             }
36
-        } catch (DotNotFoundException $e) {
40
+        }
41
+        catch (DotNotFoundException $e)
42
+        {
37 43
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
38 44
         }
39 45
 
Please login to merge, or discard this patch.
src/Config/src/ConfigManager.php 2 patches
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
     public function __construct(
23 23
         private readonly LoaderInterface $loader,
24 24
         private readonly bool $strict = true
25
-    ) {
25
+    ){
26 26
     }
27 27
 
28 28
     /**
@@ -42,11 +42,11 @@  discard block
 block discarded – undo
42 42
 
43 43
     public function setDefaults(string $section, array $data): void
44 44
     {
45
-        if (isset($this->defaults[$section])) {
45
+        if (isset($this->defaults[$section])){
46 46
             throw new ConfiguratorException(\sprintf('Unable to set default config `%s` more than once.', $section));
47 47
         }
48 48
 
49
-        if (isset($this->data[$section])) {
49
+        if (isset($this->data[$section])){
50 50
             throw new ConfigDeliveredException(
51 51
                 \sprintf('Unable to set default config `%s`, config has been loaded.', $section)
52 52
             );
@@ -57,8 +57,8 @@  discard block
 block discarded – undo
57 57
 
58 58
     public function modify(string $section, PatchInterface $patch): array
59 59
     {
60
-        if (isset($this->instances[$section])) {
61
-            if ($this->strict) {
60
+        if (isset($this->instances[$section])){
61
+            if ($this->strict){
62 62
                 throw new ConfigDeliveredException(
63 63
                     \sprintf('Unable to patch config `%s`, config object has already been delivered.', $section)
64 64
                 );
@@ -69,27 +69,27 @@  discard block
 block discarded – undo
69 69
 
70 70
         $data = $this->getConfig($section);
71 71
 
72
-        try {
72
+        try{
73 73
             return $this->data[$section] = $patch->patch($data);
74
-        } catch (PatchException $e) {
74
+        }catch (PatchException $e){
75 75
             throw new PatchException(\sprintf('Unable to modify config `%s`.', $section), $e->getCode(), $e);
76 76
         }
77 77
     }
78 78
 
79 79
     public function getConfig(string $section = null): array
80 80
     {
81
-        if (isset($this->data[$section])) {
81
+        if (isset($this->data[$section])){
82 82
             return $this->data[$section];
83 83
         }
84 84
 
85
-        if (isset($this->defaults[$section])) {
85
+        if (isset($this->defaults[$section])){
86 86
             $data = [];
87
-            if ($this->loader->has($section)) {
87
+            if ($this->loader->has($section)){
88 88
                 $data = $this->loader->load($section);
89 89
             }
90 90
 
91 91
             $data = \array_merge($this->defaults[$section], $data);
92
-        } else {
92
+        }else{
93 93
             $data = $this->loader->load($section);
94 94
         }
95 95
 
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
     public function createInjection(\ReflectionClass $class, string $context = null): object
100 100
     {
101 101
         $config = $class->getConstant('CONFIG');
102
-        if (isset($this->instances[$config])) {
102
+        if (isset($this->instances[$config])){
103 103
             return $this->instances[$config];
104 104
         }
105 105
 
Please login to merge, or discard this patch.
Braces   +24 added lines, -11 removed lines patch added patch discarded remove patch
@@ -42,11 +42,13 @@  discard block
 block discarded – undo
42 42
 
43 43
     public function setDefaults(string $section, array $data): void
44 44
     {
45
-        if (isset($this->defaults[$section])) {
45
+        if (isset($this->defaults[$section]))
46
+        {
46 47
             throw new ConfiguratorException(\sprintf('Unable to set default config `%s` more than once.', $section));
47 48
         }
48 49
 
49
-        if (isset($this->data[$section])) {
50
+        if (isset($this->data[$section]))
51
+        {
50 52
             throw new ConfigDeliveredException(
51 53
                 \sprintf('Unable to set default config `%s`, config has been loaded.', $section)
52 54
             );
@@ -57,8 +59,10 @@  discard block
 block discarded – undo
57 59
 
58 60
     public function modify(string $section, PatchInterface $patch): array
59 61
     {
60
-        if (isset($this->instances[$section])) {
61
-            if ($this->strict) {
62
+        if (isset($this->instances[$section]))
63
+        {
64
+            if ($this->strict)
65
+            {
62 66
                 throw new ConfigDeliveredException(
63 67
                     \sprintf('Unable to patch config `%s`, config object has already been delivered.', $section)
64 68
                 );
@@ -69,27 +73,35 @@  discard block
 block discarded – undo
69 73
 
70 74
         $data = $this->getConfig($section);
71 75
 
72
-        try {
76
+        try
77
+        {
73 78
             return $this->data[$section] = $patch->patch($data);
74
-        } catch (PatchException $e) {
79
+        }
80
+        catch (PatchException $e)
81
+        {
75 82
             throw new PatchException(\sprintf('Unable to modify config `%s`.', $section), $e->getCode(), $e);
76 83
         }
77 84
     }
78 85
 
79 86
     public function getConfig(string $section = null): array
80 87
     {
81
-        if (isset($this->data[$section])) {
88
+        if (isset($this->data[$section]))
89
+        {
82 90
             return $this->data[$section];
83 91
         }
84 92
 
85
-        if (isset($this->defaults[$section])) {
93
+        if (isset($this->defaults[$section]))
94
+        {
86 95
             $data = [];
87
-            if ($this->loader->has($section)) {
96
+            if ($this->loader->has($section))
97
+            {
88 98
                 $data = $this->loader->load($section);
89 99
             }
90 100
 
91 101
             $data = \array_merge($this->defaults[$section], $data);
92
-        } else {
102
+        }
103
+        else
104
+        {
93 105
             $data = $this->loader->load($section);
94 106
         }
95 107
 
@@ -99,7 +111,8 @@  discard block
 block discarded – undo
99 111
     public function createInjection(\ReflectionClass $class, string $context = null): object
100 112
     {
101 113
         $config = $class->getConstant('CONFIG');
102
-        if (isset($this->instances[$config])) {
114
+        if (isset($this->instances[$config]))
115
+        {
103 116
             return $this->instances[$config];
104 117
         }
105 118
 
Please login to merge, or discard this patch.
src/Router/tests/Loader/PhpFileLoaderTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,12 +29,12 @@
 block discarded – undo
29 29
     {
30 30
         $loader = new PhpFileLoader($this->container, $this->container);
31 31
 
32
-        $routes = $loader->load(\dirname(__DIR__) . '/Fixtures/file.php');
32
+        $routes = $loader->load(\dirname(__DIR__).'/Fixtures/file.php');
33 33
         $this->assertInstanceOf(RouteCollection::class, $routes);
34 34
         $this->assertCount(3, $routes);
35 35
 
36 36
         $this->expectException(LoaderLoadException::class);
37
-        $loader->load(\dirname(__DIR__) . '/Fixtures/unknown.php');
37
+        $loader->load(\dirname(__DIR__).'/Fixtures/unknown.php');
38 38
     }
39 39
 
40 40
     public function testSupports(): void
Please login to merge, or discard this patch.
src/Router/tests/Loader/Configurator/RoutingConfiguratorTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -17,7 +17,7 @@  discard block
 block discarded – undo
17 17
 
18 18
         $this->assertCount(0, $routes->getCollection());
19 19
 
20
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php');
20
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php');
21 21
 
22 22
         $this->assertCount(3, $routes->getCollection());
23 23
     }
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 
29 29
         $this->assertCount(0, $routes->getCollection());
30 30
 
31
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php', 'php');
31
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php', 'php');
32 32
 
33 33
         $this->assertCount(3, $routes->getCollection());
34 34
     }
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 
40 40
         $this->assertCount(0, $routes->getCollection());
41 41
 
42
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php', 'yaml');
42
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php', 'yaml');
43 43
 
44 44
         $this->assertCount(0, $routes->getCollection());
45 45
     }
Please login to merge, or discard this patch.
src/Router/tests/PipelineFactoryTest.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
             new Autowire('bar'),
72 72
         ]));
73 73
 
74
-        $handle = function (ServerRequestInterface $request, RequestHandlerInterface $handler) {
74
+        $handle = function (ServerRequestInterface $request, RequestHandlerInterface $handler){
75 75
             return $handler->handle($request);
76 76
         };
77 77
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,8 @@
 block discarded – undo
71 71
             new Autowire('bar'),
72 72
         ]));
73 73
 
74
-        $handle = function (ServerRequestInterface $request, RequestHandlerInterface $handler) {
74
+        $handle = function (ServerRequestInterface $request, RequestHandlerInterface $handler)
75
+        {
75 76
             return $handler->handle($request);
76 77
         };
77 78
 
Please login to merge, or discard this patch.
src/Router/tests/TestCore.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,6 +17,6 @@
 block discarded – undo
17 17
 
18 18
     public function callAction(string $controller, string $action = null, array $parameters = []): string
19 19
     {
20
-        return '@wrapped.' . $this->core->callAction($controller, $action, $parameters);
20
+        return '@wrapped.'.$this->core->callAction($controller, $action, $parameters);
21 21
     }
22 22
 }
Please login to merge, or discard this patch.
src/Router/tests/Fixtures/file.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@
 block discarded – undo
2 2
 
3 3
 use Spiral\Router\Loader\Configurator\RoutingConfigurator;
4 4
 
5
-return function (RoutingConfigurator $routes) {
5
+return function (RoutingConfigurator $routes){
6 6
     $routes->add('index', '/')->callable(static fn () => null);
7 7
     $routes->add('posts', '/posts')->callable(static fn () => null);
8 8
     $routes->add('post', '/post/<id>')->callable(static fn () => null);
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,8 @@
 block discarded – undo
2 2
 
3 3
 use Spiral\Router\Loader\Configurator\RoutingConfigurator;
4 4
 
5
-return function (RoutingConfigurator $routes) {
5
+return function (RoutingConfigurator $routes)
6
+{
6 7
     $routes->add('index', '/')->callable(static fn () => null);
7 8
     $routes->add('posts', '/posts')->callable(static fn () => null);
8 9
     $routes->add('post', '/post/<id>')->callable(static fn () => null);
Please login to merge, or discard this patch.
src/Router/tests/SingleActionTest.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -96,9 +96,9 @@
 block discarded – undo
96 96
         $this->assertSame('echoed', (string)$response->getBody());
97 97
 
98 98
         $e = null;
99
-        try {
99
+        try{
100 100
             $router->handle(new ServerRequest('GET', new Uri('/test')));
101
-        } catch (UndefinedRouteException $e) {
101
+        }catch (UndefinedRouteException $e){
102 102
         }
103 103
 
104 104
         $this->assertNotNull($e, 'Autofill not fired');
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -96,9 +96,12 @@
 block discarded – undo
96 96
         $this->assertSame('echoed', (string)$response->getBody());
97 97
 
98 98
         $e = null;
99
-        try {
99
+        try
100
+        {
100 101
             $router->handle(new ServerRequest('GET', new Uri('/test')));
101
-        } catch (UndefinedRouteException $e) {
102
+        }
103
+        catch (UndefinedRouteException $e)
104
+        {
102 105
         }
103 106
 
104 107
         $this->assertNotNull($e, 'Autofill not fired');
Please login to merge, or discard this patch.
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.