Passed
Pull Request — master (#1115)
by Maxim
22:29
created
src/Router/src/Target/Group.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
         private readonly array $controllers,
16 16
         int $options = 0,
17 17
         string $defaultAction = 'index'
18
-    ) {
18
+    ){
19 19
         parent::__construct(
20 20
             ['controller' => null, 'action' => null],
21 21
             ['controller' => \array_keys($controllers), 'action' => null],
Please login to merge, or discard this patch.
src/Router/src/Target/Action.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -23,16 +23,16 @@  discard block
 block discarded – undo
23 23
      */
24 24
     public function __construct(
25 25
         private readonly string $controller,
26
-        private readonly string|array $action,
26
+        private readonly string | array $action,
27 27
         int $options = 0
28
-    ) {
29
-        if (\is_string($action)) {
28
+    ){
29
+        if (\is_string($action)){
30 30
             parent::__construct(
31 31
                 ['action' => $action],
32 32
                 ['action' => new Autofill($action)],
33 33
                 $options
34 34
             );
35
-        } else {
35
+        }else{
36 36
             parent::__construct(
37 37
                 ['action' => null],
38 38
                 ['action' => $action],
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
     protected function resolveAction(array $matches): string
50 50
     {
51 51
         $action = $this->action;
52
-        if (\is_string($action)) {
52
+        if (\is_string($action)){
53 53
             return $action;
54 54
         }
55 55
 
Please login to merge, or discard this patch.
Braces   +7 added lines, -3 removed lines patch added patch discarded remove patch
@@ -26,13 +26,16 @@  discard block
 block discarded – undo
26 26
         private readonly string|array $action,
27 27
         int $options = 0
28 28
     ) {
29
-        if (\is_string($action)) {
29
+        if (\is_string($action))
30
+        {
30 31
             parent::__construct(
31 32
                 ['action' => $action],
32 33
                 ['action' => new Autofill($action)],
33 34
                 $options
34 35
             );
35
-        } else {
36
+        }
37
+        else
38
+        {
36 39
             parent::__construct(
37 40
                 ['action' => null],
38 41
                 ['action' => $action],
@@ -49,7 +52,8 @@  discard block
 block discarded – undo
49 52
     protected function resolveAction(array $matches): string
50 53
     {
51 54
         $action = $this->action;
52
-        if (\is_string($action)) {
55
+        if (\is_string($action))
56
+        {
53 57
             return $action;
54 58
         }
55 59
 
Please login to merge, or discard this patch.
src/Auth/src/Event/Logout.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -9,6 +9,6 @@
 block discarded – undo
9 9
     public function __construct(
10 10
         public readonly ?object $actor,
11 11
         public readonly ?string $transport = null
12
-    ) {
12
+    ){
13 13
     }
14 14
 }
Please login to merge, or discard this patch.
src/Auth/src/Event/Authenticated.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,6 +11,6 @@
 block discarded – undo
11 11
     public function __construct(
12 12
         public readonly TokenInterface $token,
13 13
         public readonly ?string $transport = null
14
-    ) {
14
+    ){
15 15
     }
16 16
 }
Please login to merge, or discard this patch.
src/Auth/src/AuthContext.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
     public function __construct(
19 19
         private readonly ActorProviderInterface $actorProvider,
20 20
         private readonly ?EventDispatcherInterface $eventDispatcher = null
21
-    ) {
21
+    ){
22 22
     }
23 23
 
24 24
     public function start(TokenInterface $token, string $transport = null): void
@@ -43,11 +43,11 @@  discard block
 block discarded – undo
43 43
 
44 44
     public function getActor(): ?object
45 45
     {
46
-        if ($this->closed) {
46
+        if ($this->closed){
47 47
             return null;
48 48
         }
49 49
 
50
-        if ($this->actor === null && $this->token !== null) {
50
+        if ($this->actor === null && $this->token !== null){
51 51
             $this->actor = $this->actorProvider->getActor($this->token);
52 52
         }
53 53
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,11 +43,13 @@
 block discarded – undo
43 43
 
44 44
     public function getActor(): ?object
45 45
     {
46
-        if ($this->closed) {
46
+        if ($this->closed)
47
+        {
47 48
             return null;
48 49
         }
49 50
 
50
-        if ($this->actor === null && $this->token !== null) {
51
+        if ($this->actor === null && $this->token !== null)
52
+        {
51 53
             $this->actor = $this->actorProvider->getActor($this->token);
52 54
         }
53 55
 
Please login to merge, or discard this patch.
src/Cookies/src/Config/CookiesConfig.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -40,22 +40,22 @@
 block discarded – undo
40 40
     public function resolveDomain(UriInterface $uri): ?string
41 41
     {
42 42
         $host = $uri->getHost();
43
-        if (empty($host)) {
43
+        if (empty($host)){
44 44
             return null;
45 45
         }
46 46
 
47 47
         $pattern = $this->config['domain'];
48
-        if (\preg_match("/^(\d{1,3}){4}:\d+$/", $host, $matches)) {
48
+        if (\preg_match("/^(\d{1,3}){4}:\d+$/", $host, $matches)){
49 49
             // remove port
50 50
             $host = $matches[1];
51 51
         }
52 52
 
53
-        if ($host === 'localhost' || \filter_var($host, FILTER_VALIDATE_IP)) {
53
+        if ($host === 'localhost' || \filter_var($host, FILTER_VALIDATE_IP)){
54 54
             //We can't use sub-domains when website required by IP
55 55
             $pattern = \ltrim($pattern, '.');
56 56
         }
57 57
 
58
-        if (!str_contains((string) $pattern, '%s')) {
58
+        if (!str_contains((string)$pattern, '%s')){
59 59
             //Forced domain
60 60
             return $pattern;
61 61
         }
Please login to merge, or discard this patch.
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -40,22 +40,26 @@
 block discarded – undo
40 40
     public function resolveDomain(UriInterface $uri): ?string
41 41
     {
42 42
         $host = $uri->getHost();
43
-        if (empty($host)) {
43
+        if (empty($host))
44
+        {
44 45
             return null;
45 46
         }
46 47
 
47 48
         $pattern = $this->config['domain'];
48
-        if (\preg_match("/^(\d{1,3}){4}:\d+$/", $host, $matches)) {
49
+        if (\preg_match("/^(\d{1,3}){4}:\d+$/", $host, $matches))
50
+        {
49 51
             // remove port
50 52
             $host = $matches[1];
51 53
         }
52 54
 
53
-        if ($host === 'localhost' || \filter_var($host, FILTER_VALIDATE_IP)) {
55
+        if ($host === 'localhost' || \filter_var($host, FILTER_VALIDATE_IP))
56
+        {
54 57
             //We can't use sub-domains when website required by IP
55 58
             $pattern = \ltrim($pattern, '.');
56 59
         }
57 60
 
58
-        if (!str_contains((string) $pattern, '%s')) {
61
+        if (!str_contains((string) $pattern, '%s'))
62
+        {
59 63
             //Forced domain
60 64
             return $pattern;
61 65
         }
Please login to merge, or discard this patch.
src/Cookies/src/CookieQueue.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
     public function __construct(
15 15
         private readonly ?string $domain = null,
16 16
         private readonly bool $secure = false
17
-    ) {
17
+    ){
18 18
     }
19 19
 
20 20
     /**
@@ -73,12 +73,12 @@  discard block
 block discarded – undo
73 73
         bool $httpOnly = true,
74 74
         ?string $sameSite = null
75 75
     ): self {
76
-        if (\is_null($domain)) {
76
+        if (\is_null($domain)){
77 77
             //Let's resolve domain via config
78 78
             $domain = $this->domain;
79 79
         }
80 80
 
81
-        if (\is_null($secure)) {
81
+        if (\is_null($secure)){
82 82
             $secure = $this->secure;
83 83
         }
84 84
 
@@ -102,8 +102,8 @@  discard block
 block discarded – undo
102 102
      */
103 103
     public function delete(string $name): void
104 104
     {
105
-        foreach ($this->scheduled as $index => $cookie) {
106
-            if ($cookie->getName() === $name) {
105
+        foreach ($this->scheduled as $index => $cookie){
106
+            if ($cookie->getName() === $name){
107 107
                 unset($this->scheduled[$index]);
108 108
             }
109 109
         }
Please login to merge, or discard this patch.
Braces   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -73,12 +73,14 @@  discard block
 block discarded – undo
73 73
         bool $httpOnly = true,
74 74
         ?string $sameSite = null
75 75
     ): self {
76
-        if (\is_null($domain)) {
76
+        if (\is_null($domain))
77
+        {
77 78
             //Let's resolve domain via config
78 79
             $domain = $this->domain;
79 80
         }
80 81
 
81
-        if (\is_null($secure)) {
82
+        if (\is_null($secure))
83
+        {
82 84
             $secure = $this->secure;
83 85
         }
84 86
 
@@ -102,8 +104,10 @@  discard block
 block discarded – undo
102 104
      */
103 105
     public function delete(string $name): void
104 106
     {
105
-        foreach ($this->scheduled as $index => $cookie) {
106
-            if ($cookie->getName() === $name) {
107
+        foreach ($this->scheduled as $index => $cookie)
108
+        {
109
+            if ($cookie->getName() === $name)
110
+            {
107 111
                 unset($this->scheduled[$index]);
108 112
             }
109 113
         }
Please login to merge, or discard this patch.
src/Cookies/src/Middleware/CookiesMiddleware.php 2 patches
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
     public function __construct(
26 26
         private readonly CookiesConfig $config,
27 27
         private readonly EncryptionInterface $encryption
28
-    ) {
28
+    ){
29 29
     }
30 30
 
31 31
     public function process(Request $request, RequestHandlerInterface $handler): Response
@@ -50,8 +50,8 @@  discard block
 block discarded – undo
50 50
     {
51 51
         $cookies = $request->getCookieParams();
52 52
 
53
-        foreach ($cookies as $name => $cookie) {
54
-            if (!$this->isProtected($name)) {
53
+        foreach ($cookies as $name => $cookie){
54
+            if (!$this->isProtected($name)){
55 55
                 continue;
56 56
             }
57 57
 
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
      */
67 67
     protected function isProtected(string $cookie): bool
68 68
     {
69
-        if (\in_array($cookie, $this->config->getExcludedCookies(), true)) {
69
+        if (\in_array($cookie, $this->config->getExcludedCookies(), true)){
70 70
             //Excluded
71 71
             return false;
72 72
         }
@@ -81,14 +81,14 @@  discard block
 block discarded – undo
81 81
      */
82 82
     protected function packCookies(Response $response, CookieQueue $queue): Response
83 83
     {
84
-        if (empty($queue->getScheduled())) {
84
+        if (empty($queue->getScheduled())){
85 85
             return $response;
86 86
         }
87 87
 
88 88
         $cookies = $response->getHeader('Set-Cookie');
89 89
 
90
-        foreach ($queue->getScheduled() as $cookie) {
91
-            if (empty($cookie->getValue()) || !$this->isProtected($cookie->getName())) {
90
+        foreach ($queue->getScheduled() as $cookie){
91
+            if (empty($cookie->getValue()) || !$this->isProtected($cookie->getName())){
92 92
                 $cookies[] = $cookie->createHeader();
93 93
                 continue;
94 94
             }
@@ -99,28 +99,28 @@  discard block
 block discarded – undo
99 99
         return $response->withHeader('Set-Cookie', $cookies);
100 100
     }
101 101
 
102
-    private function decodeCookie(array|string $cookie): mixed
102
+    private function decodeCookie(array | string $cookie): mixed
103 103
     {
104
-        try {
105
-            if (\is_array($cookie)) {
106
-                return \array_map(fn (array|string $cookie) => $this->decodeCookie($cookie), $cookie);
104
+        try{
105
+            if (\is_array($cookie)){
106
+                return \array_map(fn (array | string $cookie) => $this->decodeCookie($cookie), $cookie);
107 107
             }
108
-        } catch (DecryptException) {
108
+        }catch (DecryptException){
109 109
             return null;
110 110
         }
111 111
 
112
-        switch ($this->config->getProtectionMethod()) {
112
+        switch ($this->config->getProtectionMethod()){
113 113
             case CookiesConfig::COOKIE_ENCRYPT:
114
-                try {
114
+                try{
115 115
                     return $this->encryption->getEncrypter()->decrypt($cookie);
116
-                } catch (DecryptException) {
116
+                }catch (DecryptException){
117 117
                 }
118 118
                 return null;
119 119
             case CookiesConfig::COOKIE_HMAC:
120 120
                 $hmac = \substr($cookie, -1 * CookiesConfig::MAC_LENGTH);
121 121
                 $value = \substr($cookie, 0, \strlen($cookie) - \strlen($hmac));
122 122
 
123
-                if (\hash_equals($this->hmacSign($value), $hmac)) {
123
+                if (\hash_equals($this->hmacSign($value), $hmac)){
124 124
                     return $value;
125 125
                 }
126 126
         }
@@ -143,13 +143,13 @@  discard block
 block discarded – undo
143 143
     private function encodeCookie(Cookie $cookie): Cookie
144 144
     {
145 145
         $value = $cookie->getValue() ?? '';
146
-        if ($this->config->getProtectionMethod() === CookiesConfig::COOKIE_ENCRYPT) {
146
+        if ($this->config->getProtectionMethod() === CookiesConfig::COOKIE_ENCRYPT){
147 147
             $encryptor = $this->encryption->getEncrypter();
148 148
 
149 149
             return $cookie->withValue($encryptor->encrypt($value));
150 150
         }
151 151
 
152 152
         //VALUE.HMAC
153
-        return $cookie->withValue($value . $this->hmacSign($value));
153
+        return $cookie->withValue($value.$this->hmacSign($value));
154 154
     }
155 155
 }
Please login to merge, or discard this patch.
Braces   +30 added lines, -14 removed lines patch added patch discarded remove patch
@@ -50,8 +50,10 @@  discard block
 block discarded – undo
50 50
     {
51 51
         $cookies = $request->getCookieParams();
52 52
 
53
-        foreach ($cookies as $name => $cookie) {
54
-            if (!$this->isProtected($name)) {
53
+        foreach ($cookies as $name => $cookie)
54
+        {
55
+            if (!$this->isProtected($name))
56
+            {
55 57
                 continue;
56 58
             }
57 59
 
@@ -66,7 +68,8 @@  discard block
 block discarded – undo
66 68
      */
67 69
     protected function isProtected(string $cookie): bool
68 70
     {
69
-        if (\in_array($cookie, $this->config->getExcludedCookies(), true)) {
71
+        if (\in_array($cookie, $this->config->getExcludedCookies(), true))
72
+        {
70 73
             //Excluded
71 74
             return false;
72 75
         }
@@ -81,14 +84,17 @@  discard block
 block discarded – undo
81 84
      */
82 85
     protected function packCookies(Response $response, CookieQueue $queue): Response
83 86
     {
84
-        if (empty($queue->getScheduled())) {
87
+        if (empty($queue->getScheduled()))
88
+        {
85 89
             return $response;
86 90
         }
87 91
 
88 92
         $cookies = $response->getHeader('Set-Cookie');
89 93
 
90
-        foreach ($queue->getScheduled() as $cookie) {
91
-            if (empty($cookie->getValue()) || !$this->isProtected($cookie->getName())) {
94
+        foreach ($queue->getScheduled() as $cookie)
95
+        {
96
+            if (empty($cookie->getValue()) || !$this->isProtected($cookie->getName()))
97
+            {
92 98
                 $cookies[] = $cookie->createHeader();
93 99
                 continue;
94 100
             }
@@ -101,26 +107,35 @@  discard block
 block discarded – undo
101 107
 
102 108
     private function decodeCookie(array|string $cookie): mixed
103 109
     {
104
-        try {
105
-            if (\is_array($cookie)) {
110
+        try
111
+        {
112
+            if (\is_array($cookie))
113
+            {
106 114
                 return \array_map(fn (array|string $cookie) => $this->decodeCookie($cookie), $cookie);
107 115
             }
108
-        } catch (DecryptException) {
116
+        }
117
+        catch (DecryptException)
118
+        {
109 119
             return null;
110 120
         }
111 121
 
112
-        switch ($this->config->getProtectionMethod()) {
122
+        switch ($this->config->getProtectionMethod())
123
+        {
113 124
             case CookiesConfig::COOKIE_ENCRYPT:
114
-                try {
125
+                try
126
+                {
115 127
                     return $this->encryption->getEncrypter()->decrypt($cookie);
116
-                } catch (DecryptException) {
128
+                }
129
+                catch (DecryptException)
130
+                {
117 131
                 }
118 132
                 return null;
119 133
             case CookiesConfig::COOKIE_HMAC:
120 134
                 $hmac = \substr($cookie, -1 * CookiesConfig::MAC_LENGTH);
121 135
                 $value = \substr($cookie, 0, \strlen($cookie) - \strlen($hmac));
122 136
 
123
-                if (\hash_equals($this->hmacSign($value), $hmac)) {
137
+                if (\hash_equals($this->hmacSign($value), $hmac))
138
+                {
124 139
                     return $value;
125 140
                 }
126 141
         }
@@ -143,7 +158,8 @@  discard block
 block discarded – undo
143 158
     private function encodeCookie(Cookie $cookie): Cookie
144 159
     {
145 160
         $value = $cookie->getValue() ?? '';
146
-        if ($this->config->getProtectionMethod() === CookiesConfig::COOKIE_ENCRYPT) {
161
+        if ($this->config->getProtectionMethod() === CookiesConfig::COOKIE_ENCRYPT)
162
+        {
147 163
             $encryptor = $this->encryption->getEncrypter();
148 164
 
149 165
             return $cookie->withValue($encryptor->encrypt($value));
Please login to merge, or discard this patch.
src/Cookies/src/Cookie.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
         private readonly bool $secure = false,
63 63
         private readonly bool $httpOnly = true,
64 64
         ?string $sameSite = null
65
-    ) {
65
+    ){
66 66
         $this->sameSite = new Cookie\SameSite($sameSite, $secure);
67 67
     }
68 68
 
@@ -158,30 +158,30 @@  discard block
 block discarded – undo
158 158
      */
159 159
     public function createHeader(): string
160 160
     {
161
-        $header = [\rawurlencode($this->name) . '=' . \rawurlencode((string)$this->value)];
161
+        $header = [\rawurlencode($this->name).'='.\rawurlencode((string)$this->value)];
162 162
 
163
-        if ($this->lifetime !== null) {
164
-            $header[] = 'Expires=' . \gmdate(\DateTime::COOKIE, $this->getExpires());
163
+        if ($this->lifetime !== null){
164
+            $header[] = 'Expires='.\gmdate(\DateTime::COOKIE, $this->getExpires());
165 165
             $header[] = \sprintf('Max-Age=%d', $this->lifetime);
166 166
         }
167 167
 
168
-        if (!empty($this->path)) {
168
+        if (!empty($this->path)){
169 169
             $header[] = \sprintf('Path=%s', $this->path);
170 170
         }
171 171
 
172
-        if (!empty($this->domain)) {
172
+        if (!empty($this->domain)){
173 173
             $header[] = \sprintf('Domain=%s', $this->domain);
174 174
         }
175 175
 
176
-        if ($this->secure) {
176
+        if ($this->secure){
177 177
             $header[] = 'Secure';
178 178
         }
179 179
 
180
-        if ($this->httpOnly) {
180
+        if ($this->httpOnly){
181 181
             $header[] = 'HttpOnly';
182 182
         }
183 183
 
184
-        if ($this->sameSite->get() !== null) {
184
+        if ($this->sameSite->get() !== null){
185 185
             $header[] = \sprintf('SameSite=%s', $this->sameSite->get());
186 186
         }
187 187
 
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
195 195
      */
196 196
     public function getExpires(): ?int
197 197
     {
198
-        if ($this->lifetime === null) {
198
+        if ($this->lifetime === null){
199 199
             return null;
200 200
         }
201 201
 
Please login to merge, or discard this patch.
Braces   +14 added lines, -7 removed lines patch added patch discarded remove patch
@@ -160,28 +160,34 @@  discard block
 block discarded – undo
160 160
     {
161 161
         $header = [\rawurlencode($this->name) . '=' . \rawurlencode((string)$this->value)];
162 162
 
163
-        if ($this->lifetime !== null) {
163
+        if ($this->lifetime !== null)
164
+        {
164 165
             $header[] = 'Expires=' . \gmdate(\DateTime::COOKIE, $this->getExpires());
165 166
             $header[] = \sprintf('Max-Age=%d', $this->lifetime);
166 167
         }
167 168
 
168
-        if (!empty($this->path)) {
169
+        if (!empty($this->path))
170
+        {
169 171
             $header[] = \sprintf('Path=%s', $this->path);
170 172
         }
171 173
 
172
-        if (!empty($this->domain)) {
174
+        if (!empty($this->domain))
175
+        {
173 176
             $header[] = \sprintf('Domain=%s', $this->domain);
174 177
         }
175 178
 
176
-        if ($this->secure) {
179
+        if ($this->secure)
180
+        {
177 181
             $header[] = 'Secure';
178 182
         }
179 183
 
180
-        if ($this->httpOnly) {
184
+        if ($this->httpOnly)
185
+        {
181 186
             $header[] = 'HttpOnly';
182 187
         }
183 188
 
184
-        if ($this->sameSite->get() !== null) {
189
+        if ($this->sameSite->get() !== null)
190
+        {
185 191
             $header[] = \sprintf('SameSite=%s', $this->sameSite->get());
186 192
         }
187 193
 
@@ -195,7 +201,8 @@  discard block
 block discarded – undo
195 201
      */
196 202
     public function getExpires(): ?int
197 203
     {
198
-        if ($this->lifetime === null) {
204
+        if ($this->lifetime === null)
205
+        {
199 206
             return null;
200 207
         }
201 208
 
Please login to merge, or discard this patch.