Passed
Pull Request — master (#1059)
by Maxim
15:44 queued 04:26
created
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/Session/Middleware/SessionMiddleware.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
         private readonly CookiesConfig $cookiesConfig,
32 32
         private readonly SessionFactoryInterface $factory,
33 33
         private readonly ScopeInterface $scope
34
-    ) {
34
+    ){
35 35
     }
36 36
 
37 37
     /**
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
             $this->fetchID($request)
46 46
         );
47 47
 
48
-        try {
48
+        try{
49 49
             $response = $this->scope->runScope(
50 50
                 [SessionInterface::class => $session],
51 51
                 static fn () => $handler->handle($request->withAttribute(self::ATTRIBUTE, $session))
52 52
             );
53
-        } catch (\Throwable $e) {
53
+        }catch (\Throwable $e){
54 54
             $session->abort();
55 55
             throw $e;
56 56
         }
@@ -63,14 +63,14 @@  discard block
 block discarded – undo
63 63
         Request $request,
64 64
         Response $response
65 65
     ): Response {
66
-        if (!$session->isStarted()) {
66
+        if (!$session->isStarted()){
67 67
             return $response;
68 68
         }
69 69
 
70 70
         $session->commit();
71 71
 
72 72
         //SID changed
73
-        if ($this->fetchID($request) !== $session->getID()) {
73
+        if ($this->fetchID($request) !== $session->getID()){
74 74
             return $this->withCookie($request, $response, $session->getID());
75 75
         }
76 76
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
     protected function fetchID(Request $request): ?string
85 85
     {
86 86
         $cookies = $request->getCookieParams();
87
-        if (empty($cookies[$this->config->getCookie()])) {
87
+        if (empty($cookies[$this->config->getCookie()])){
88 88
             return null;
89 89
         }
90 90
 
@@ -106,8 +106,8 @@  discard block
 block discarded – undo
106 106
     protected function clientSignature(Request $request): string
107 107
     {
108 108
         $signature = '';
109
-        foreach (self::SIGNATURE_HEADERS as $header) {
110
-            $signature .= $request->getHeaderLine($header) . ';';
109
+        foreach (self::SIGNATURE_HEADERS as $header){
110
+            $signature .= $request->getHeaderLine($header).';';
111 111
         }
112 112
 
113 113
         return \hash('sha256', $signature);
Please login to merge, or discard this patch.
Braces   +13 added lines, -6 removed lines patch added patch discarded remove patch
@@ -45,12 +45,15 @@  discard block
 block discarded – undo
45 45
             $this->fetchID($request)
46 46
         );
47 47
 
48
-        try {
48
+        try
49
+        {
49 50
             $response = $this->scope->runScope(
50 51
                 [SessionInterface::class => $session],
51 52
                 static fn () => $handler->handle($request->withAttribute(self::ATTRIBUTE, $session))
52 53
             );
53
-        } catch (\Throwable $e) {
54
+        }
55
+        catch (\Throwable $e)
56
+        {
54 57
             $session->abort();
55 58
             throw $e;
56 59
         }
@@ -63,14 +66,16 @@  discard block
 block discarded – undo
63 66
         Request $request,
64 67
         Response $response
65 68
     ): Response {
66
-        if (!$session->isStarted()) {
69
+        if (!$session->isStarted())
70
+        {
67 71
             return $response;
68 72
         }
69 73
 
70 74
         $session->commit();
71 75
 
72 76
         //SID changed
73
-        if ($this->fetchID($request) !== $session->getID()) {
77
+        if ($this->fetchID($request) !== $session->getID())
78
+        {
74 79
             return $this->withCookie($request, $response, $session->getID());
75 80
         }
76 81
 
@@ -84,7 +89,8 @@  discard block
 block discarded – undo
84 89
     protected function fetchID(Request $request): ?string
85 90
     {
86 91
         $cookies = $request->getCookieParams();
87
-        if (empty($cookies[$this->config->getCookie()])) {
92
+        if (empty($cookies[$this->config->getCookie()]))
93
+        {
88 94
             return null;
89 95
         }
90 96
 
@@ -106,7 +112,8 @@  discard block
 block discarded – undo
106 112
     protected function clientSignature(Request $request): string
107 113
     {
108 114
         $signature = '';
109
-        foreach (self::SIGNATURE_HEADERS as $header) {
115
+        foreach (self::SIGNATURE_HEADERS as $header)
116
+        {
110 117
             $signature .= $request->getHeaderLine($header) . ';';
111 118
         }
112 119
 
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
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
 {
16 16
     public function __construct(
17 17
         private readonly ContainerInterface $container
18
-    ) {
18
+    ){
19 19
     }
20 20
 
21 21
     /**
@@ -68,9 +68,9 @@  discard block
 block discarded – undo
68 68
 
69 69
     private function getAuthContext(): AuthContextInterface
70 70
     {
71
-        try {
71
+        try{
72 72
             return $this->container->get(AuthContextInterface::class);
73
-        } catch (NotFoundExceptionInterface $e) {
73
+        }catch (NotFoundExceptionInterface $e){
74 74
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
75 75
         }
76 76
     }
Please login to merge, or discard this patch.
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.
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.