Passed
Pull Request — master (#1136)
by Abdul Malik
11:17
created
src/Reactor/src/FileDeclaration.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,8 @@
 block discarded – undo
63 63
 
64 64
     public function addNamespace(string|PhpNamespace $namespace): PhpNamespace
65 65
     {
66
-        if ($namespace instanceof PhpNamespace) {
66
+        if ($namespace instanceof PhpNamespace)
67
+        {
67 68
             $this->element->addNamespace($namespace->getElement());
68 69
 
69 70
             return $namespace;
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -53,9 +53,9 @@
 block discarded – undo
53 53
         return self::fromElement((new Factory())->fromCode(\file_get_contents($reflection->getFileName())));
54 54
     }
55 55
 
56
-    public function addNamespace(string|PhpNamespace $namespace): PhpNamespace
56
+    public function addNamespace(string | PhpNamespace $namespace): PhpNamespace
57 57
     {
58
-        if ($namespace instanceof PhpNamespace) {
58
+        if ($namespace instanceof PhpNamespace){
59 59
             $this->element->addNamespace($namespace->getElement());
60 60
 
61 61
             return $namespace;
Please login to merge, or discard this patch.
src/Validation/tests/ValidationProviderTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@
 block discarded – undo
11 11
 
12 12
 final class ValidationProviderTest extends TestCase
13 13
 {
14
-    private m\MockInterface|InvokerInterface $invoker;
14
+    private m\MockInterface | InvokerInterface $invoker;
15 15
     private ValidationProvider $provider;
16 16
 
17 17
     protected function setUp(): void
Please login to merge, or discard this patch.
src/Validation/src/ValidatorInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@
 block discarded – undo
14 14
     /**
15 15
      * Create validator copy with new data set.
16 16
      */
17
-    public function withData(array|object $data): ValidatorInterface;
17
+    public function withData(array | object $data): ValidatorInterface;
18 18
 
19 19
     /**
20 20
      * Receive field from context data or return default value.
Please login to merge, or discard this patch.
src/Validation/src/ValidationInterface.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -17,5 +17,5 @@
 block discarded – undo
17 17
      * @param mixed         $context Validation context (available for checkers and validation
18 18
      *                               methods but is not validated).
19 19
      */
20
-    public function validate(array|object $data, array $rules, mixed $context = null): ValidatorInterface;
20
+    public function validate(array | object $data, array $rules, mixed $context = null): ValidatorInterface;
21 21
 }
Please login to merge, or discard this patch.
src/Framework/Session/SectionScope.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,7 +9,7 @@
 block discarded – undo
9 9
     public function __construct(
10 10
         private readonly SessionScope $session,
11 11
         private readonly string $name
12
-    ) {
12
+    ){
13 13
     }
14 14
 
15 15
     /**
Please login to merge, or discard this patch.
src/Framework/Auth/Session/Token.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -11,8 +11,8 @@  discard block
 block discarded – undo
11 11
     public function __construct(
12 12
         private readonly string $id,
13 13
         private readonly array $payload,
14
-        private readonly ?\DateTimeInterface $expiresAt = null
15
-    ) {
14
+        private readonly ? \DateTimeInterface $expiresAt = null
15
+    ){
16 16
     }
17 17
 
18 18
     public function getID(): string
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
     public static function unpack(array $data): Token
51 51
     {
52 52
         $expiresAt = null;
53
-        if ($data['expiresAt'] !== null) {
53
+        if ($data['expiresAt'] !== null){
54 54
             $expiresAt = (new \DateTimeImmutable())->setTimestamp($data['expiresAt']);
55 55
         }
56 56
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,8 @@
 block discarded – undo
50 50
     public static function unpack(array $data): Token
51 51
     {
52 52
         $expiresAt = null;
53
-        if ($data['expiresAt'] !== null) {
53
+        if ($data['expiresAt'] !== null)
54
+        {
54 55
             $expiresAt = (new \DateTimeImmutable())->setTimestamp($data['expiresAt']);
55 56
         }
56 57
 
Please login to merge, or discard this patch.
src/Framework/Auth/Session/TokenStorage.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -19,28 +19,28 @@  discard block
 block discarded – undo
19 19
 
20 20
     public function __construct(
21 21
         private readonly SessionScope $session
22
-    ) {
22
+    ){
23 23
     }
24 24
 
25 25
     public function load(string $id): ?TokenInterface
26 26
     {
27
-        try {
27
+        try{
28 28
             $tokenData = $this->session->getSection(self::SESSION_SECTION)->get('token');
29
-            if ($tokenData === null) {
29
+            if ($tokenData === null){
30 30
                 return null;
31 31
             }
32 32
 
33 33
             $token = Token::unpack($tokenData);
34
-        } catch (\Throwable $e) {
35
-            throw new TokenStorageException('Unable to load session token', (int) $e->getCode(), $e);
34
+        }catch (\Throwable $e){
35
+            throw new TokenStorageException('Unable to load session token', (int)$e->getCode(), $e);
36 36
         }
37 37
 
38
-        if (!\hash_equals($token->getID(), $id)) {
38
+        if (!\hash_equals($token->getID(), $id)){
39 39
             return null;
40 40
         }
41 41
 
42 42
         $expiresAt = $token->getExpiresAt();
43
-        if ($expiresAt !== null && $expiresAt < new \DateTimeImmutable()) {
43
+        if ($expiresAt !== null && $expiresAt < new \DateTimeImmutable()){
44 44
             $this->delete($token);
45 45
             return null;
46 46
         }
@@ -50,13 +50,13 @@  discard block
 block discarded – undo
50 50
 
51 51
     public function create(array $payload, \DateTimeInterface $expiresAt = null): TokenInterface
52 52
     {
53
-        try {
53
+        try{
54 54
             $token = new Token($this->randomHash(128), $payload, $expiresAt);
55 55
             $this->session->getSection(self::SESSION_SECTION)->set('token', $token->pack());
56 56
 
57 57
             return $token;
58
-        } catch (\Throwable $e) {
59
-            throw new TokenStorageException('Unable to create session token', (int) $e->getCode(), $e);
58
+        }catch (\Throwable $e){
59
+            throw new TokenStorageException('Unable to create session token', (int)$e->getCode(), $e);
60 60
         }
61 61
     }
62 62
 
Please login to merge, or discard this patch.
Braces   +16 added lines, -7 removed lines patch added patch discarded remove patch
@@ -24,23 +24,29 @@  discard block
 block discarded – undo
24 24
 
25 25
     public function load(string $id): ?TokenInterface
26 26
     {
27
-        try {
27
+        try
28
+        {
28 29
             $tokenData = $this->session->getSection(self::SESSION_SECTION)->get('token');
29
-            if ($tokenData === null) {
30
+            if ($tokenData === null)
31
+            {
30 32
                 return null;
31 33
             }
32 34
 
33 35
             $token = Token::unpack($tokenData);
34
-        } catch (\Throwable $e) {
36
+        }
37
+        catch (\Throwable $e)
38
+        {
35 39
             throw new TokenStorageException('Unable to load session token', (int) $e->getCode(), $e);
36 40
         }
37 41
 
38
-        if (!\hash_equals($token->getID(), $id)) {
42
+        if (!\hash_equals($token->getID(), $id))
43
+        {
39 44
             return null;
40 45
         }
41 46
 
42 47
         $expiresAt = $token->getExpiresAt();
43
-        if ($expiresAt !== null && $expiresAt < new \DateTimeImmutable()) {
48
+        if ($expiresAt !== null && $expiresAt < new \DateTimeImmutable())
49
+        {
44 50
             $this->delete($token);
45 51
             return null;
46 52
         }
@@ -50,12 +56,15 @@  discard block
 block discarded – undo
50 56
 
51 57
     public function create(array $payload, \DateTimeInterface $expiresAt = null): TokenInterface
52 58
     {
53
-        try {
59
+        try
60
+        {
54 61
             $token = new Token($this->randomHash(128), $payload, $expiresAt);
55 62
             $this->session->getSection(self::SESSION_SECTION)->set('token', $token->pack());
56 63
 
57 64
             return $token;
58
-        } catch (\Throwable $e) {
65
+        }
66
+        catch (\Throwable $e)
67
+        {
59 68
             throw new TokenStorageException('Unable to create session token', (int) $e->getCode(), $e);
60 69
         }
61 70
     }
Please login to merge, or discard this patch.
src/Framework/Auth/AuthScope.php 2 patches
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -68,9 +68,12 @@
 block discarded – undo
68 68
 
69 69
     private function getAuthContext(): AuthContextInterface
70 70
     {
71
-        try {
71
+        try
72
+        {
72 73
             return $this->container->get(AuthContextInterface::class);
73
-        } catch (NotFoundExceptionInterface $e) {
74
+        }
75
+        catch (NotFoundExceptionInterface $e)
76
+        {
74 77
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
75 78
         }
76 79
     }
Please login to merge, or discard this 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
     public function __construct(
19 19
         #[Proxy] private readonly ContainerInterface $container
20
-    ) {
20
+    ){
21 21
     }
22 22
 
23 23
     /**
@@ -70,9 +70,9 @@  discard block
 block discarded – undo
70 70
 
71 71
     private function getAuthContext(): AuthContextInterface
72 72
     {
73
-        try {
73
+        try{
74 74
             return $this->container->get(AuthContextInterface::class);
75
-        } catch (NotFoundExceptionInterface $e) {
75
+        }catch (NotFoundExceptionInterface $e){
76 76
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
77 77
         }
78 78
     }
Please login to merge, or discard this patch.
src/Framework/Domain/PipelineInterceptor.php 2 patches
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
         private ReaderInterface $reader,
22 22
         private ContainerInterface $container,
23 23
         private ?EventDispatcherInterface $dispatcher = null
24
-    ) {
24
+    ){
25 25
     }
26 26
 
27 27
     /**
@@ -30,17 +30,17 @@  discard block
 block discarded – undo
30 30
     public function process(string $controller, string $action, array $parameters, CoreInterface $core): mixed
31 31
     {
32 32
         $annotation = $this->readAnnotation($controller, $action);
33
-        if ($core instanceof InterceptorPipeline && $annotation->skipNext) {
33
+        if ($core instanceof InterceptorPipeline && $annotation->skipNext){
34 34
             $this->removeNextInterceptorsFromOriginalPipeline($core);
35 35
         }
36 36
 
37 37
         $pipeline = $this->getCachedPipeline($controller, $action, $annotation);
38
-        if (!empty($pipeline)) {
39
-            if ($core instanceof InterceptorPipeline) {
38
+        if (!empty($pipeline)){
39
+            if ($core instanceof InterceptorPipeline){
40 40
                 $this->injectInterceptorsIntoOriginalPipeline($core, $pipeline);
41
-            } else {
41
+            }else{
42 42
                 $core = new InterceptableCore($core, $this->dispatcher);
43
-                foreach ($pipeline as $interceptor) {
43
+                foreach ($pipeline as $interceptor){
44 44
                     $core->addInterceptor($interceptor);
45 45
                 }
46 46
             }
@@ -51,9 +51,9 @@  discard block
 block discarded – undo
51 51
 
52 52
     private function readAnnotation(string $controller, string $action): Pipeline
53 53
     {
54
-        try {
54
+        try{
55 55
             $method = new \ReflectionMethod($controller, $action);
56
-        } catch (\ReflectionException) {
56
+        }catch (\ReflectionException){
57 57
             return new Pipeline();
58 58
         }
59 59
 
@@ -68,14 +68,14 @@  discard block
 block discarded – undo
68 68
 
69 69
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
70 70
         $newInterceptors = [];
71
-        foreach ($oldInterceptors as $interceptor) {
71
+        foreach ($oldInterceptors as $interceptor){
72 72
             $newInterceptors[] = $interceptor;
73
-            if ($interceptor instanceof self) {
73
+            if ($interceptor instanceof self){
74 74
                 break;
75 75
             }
76 76
         }
77 77
 
78
-        if (\count($newInterceptors) !== \count($oldInterceptors)) {
78
+        if (\count($newInterceptors) !== \count($oldInterceptors)){
79 79
             $pipelineReflection->setValue($pipeline, $newInterceptors);
80 80
         }
81 81
     }
@@ -86,16 +86,16 @@  discard block
 block discarded – undo
86 86
 
87 87
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
88 88
         $newInterceptors = [];
89
-        foreach ($oldInterceptors as $interceptor) {
89
+        foreach ($oldInterceptors as $interceptor){
90 90
             $newInterceptors[] = $interceptor;
91
-            if ($interceptor instanceof self) {
92
-                foreach ($interceptors as $newInterceptor) {
91
+            if ($interceptor instanceof self){
92
+                foreach ($interceptors as $newInterceptor){
93 93
                     $newInterceptors[] = $newInterceptor;
94 94
                 }
95 95
             }
96 96
         }
97 97
 
98
-        if (\count($newInterceptors) !== \count($oldInterceptors)) {
98
+        if (\count($newInterceptors) !== \count($oldInterceptors)){
99 99
             $pipelineReflection->setValue($pipeline, $newInterceptors);
100 100
         }
101 101
     }
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
     private function getCachedPipeline(string $controller, string $action, Pipeline $annotation): array
104 104
     {
105 105
         $key = "{$controller}:{$action}";
106
-        if (!\array_key_exists($key, $this->cache)) {
106
+        if (!\array_key_exists($key, $this->cache)){
107 107
             $this->cache[$key] = $this->extractAnnotationPipeline($annotation);
108 108
         }
109 109
 
@@ -113,10 +113,10 @@  discard block
 block discarded – undo
113 113
     private function extractAnnotationPipeline(Pipeline $annotation): array
114 114
     {
115 115
         $interceptors = [];
116
-        foreach ($annotation->pipeline as $interceptor) {
117
-            try {
116
+        foreach ($annotation->pipeline as $interceptor){
117
+            try{
118 118
                 $interceptors[] = $this->container->get($interceptor);
119
-            } catch (\Throwable) {
119
+            }catch (\Throwable){
120 120
             }
121 121
         }
122 122
 
Please login to merge, or discard this patch.
Braces   +39 added lines, -18 removed lines patch added patch discarded remove patch
@@ -30,17 +30,23 @@  discard block
 block discarded – undo
30 30
     public function process(string $controller, string $action, array $parameters, CoreInterface $core): mixed
31 31
     {
32 32
         $annotation = $this->readAnnotation($controller, $action);
33
-        if ($core instanceof InterceptorPipeline && $annotation->skipNext) {
33
+        if ($core instanceof InterceptorPipeline && $annotation->skipNext)
34
+        {
34 35
             $this->removeNextInterceptorsFromOriginalPipeline($core);
35 36
         }
36 37
 
37 38
         $pipeline = $this->getCachedPipeline($controller, $action, $annotation);
38
-        if (!empty($pipeline)) {
39
-            if ($core instanceof InterceptorPipeline) {
39
+        if (!empty($pipeline))
40
+        {
41
+            if ($core instanceof InterceptorPipeline)
42
+            {
40 43
                 $this->injectInterceptorsIntoOriginalPipeline($core, $pipeline);
41
-            } else {
44
+            }
45
+            else
46
+            {
42 47
                 $core = new InterceptableCore($core, $this->dispatcher);
43
-                foreach ($pipeline as $interceptor) {
48
+                foreach ($pipeline as $interceptor)
49
+                {
44 50
                     $core->addInterceptor($interceptor);
45 51
                 }
46 52
             }
@@ -51,9 +57,12 @@  discard block
 block discarded – undo
51 57
 
52 58
     private function readAnnotation(string $controller, string $action): Pipeline
53 59
     {
54
-        try {
60
+        try
61
+        {
55 62
             $method = new \ReflectionMethod($controller, $action);
56
-        } catch (\ReflectionException) {
63
+        }
64
+        catch (\ReflectionException)
65
+        {
57 66
             return new Pipeline();
58 67
         }
59 68
 
@@ -68,14 +77,17 @@  discard block
 block discarded – undo
68 77
 
69 78
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
70 79
         $newInterceptors = [];
71
-        foreach ($oldInterceptors as $interceptor) {
80
+        foreach ($oldInterceptors as $interceptor)
81
+        {
72 82
             $newInterceptors[] = $interceptor;
73
-            if ($interceptor instanceof self) {
83
+            if ($interceptor instanceof self)
84
+            {
74 85
                 break;
75 86
             }
76 87
         }
77 88
 
78
-        if (\count($newInterceptors) !== \count($oldInterceptors)) {
89
+        if (\count($newInterceptors) !== \count($oldInterceptors))
90
+        {
79 91
             $pipelineReflection->setValue($pipeline, $newInterceptors);
80 92
         }
81 93
     }
@@ -86,16 +98,20 @@  discard block
 block discarded – undo
86 98
 
87 99
         $oldInterceptors = $pipelineReflection->getValue($pipeline);
88 100
         $newInterceptors = [];
89
-        foreach ($oldInterceptors as $interceptor) {
101
+        foreach ($oldInterceptors as $interceptor)
102
+        {
90 103
             $newInterceptors[] = $interceptor;
91
-            if ($interceptor instanceof self) {
92
-                foreach ($interceptors as $newInterceptor) {
104
+            if ($interceptor instanceof self)
105
+            {
106
+                foreach ($interceptors as $newInterceptor)
107
+                {
93 108
                     $newInterceptors[] = $newInterceptor;
94 109
                 }
95 110
             }
96 111
         }
97 112
 
98
-        if (\count($newInterceptors) !== \count($oldInterceptors)) {
113
+        if (\count($newInterceptors) !== \count($oldInterceptors))
114
+        {
99 115
             $pipelineReflection->setValue($pipeline, $newInterceptors);
100 116
         }
101 117
     }
@@ -103,7 +119,8 @@  discard block
 block discarded – undo
103 119
     private function getCachedPipeline(string $controller, string $action, Pipeline $annotation): array
104 120
     {
105 121
         $key = "{$controller}:{$action}";
106
-        if (!\array_key_exists($key, $this->cache)) {
122
+        if (!\array_key_exists($key, $this->cache))
123
+        {
107 124
             $this->cache[$key] = $this->extractAnnotationPipeline($annotation);
108 125
         }
109 126
 
@@ -113,10 +130,14 @@  discard block
 block discarded – undo
113 130
     private function extractAnnotationPipeline(Pipeline $annotation): array
114 131
     {
115 132
         $interceptors = [];
116
-        foreach ($annotation->pipeline as $interceptor) {
117
-            try {
133
+        foreach ($annotation->pipeline as $interceptor)
134
+        {
135
+            try
136
+            {
118 137
                 $interceptors[] = $this->container->get($interceptor);
119
-            } catch (\Throwable) {
138
+            }
139
+            catch (\Throwable)
140
+            {
120 141
             }
121 142
         }
122 143
 
Please login to merge, or discard this patch.