Passed
Pull Request — master (#225)
by Anton
02:54
created
src/Session/SessionScope.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -123,9 +123,9 @@
 block discarded – undo
123 123
      */
124 124
     public function getActiveSession(): SessionInterface
125 125
     {
126
-        try {
126
+        try{
127 127
             return $this->container->get(SessionInterface::class);
128
-        } catch (NotFoundExceptionInterface $e) {
128
+        }catch (NotFoundExceptionInterface $e){
129 129
             throw new ScopeException('Unable to receive active session, invalid request scope', $e->getCode(), $e);
130 130
         }
131 131
     }
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -123,9 +123,12 @@
 block discarded – undo
123 123
      */
124 124
     public function getActiveSession(): SessionInterface
125 125
     {
126
-        try {
126
+        try
127
+        {
127 128
             return $this->container->get(SessionInterface::class);
128
-        } catch (NotFoundExceptionInterface $e) {
129
+        }
130
+        catch (NotFoundExceptionInterface $e)
131
+        {
129 132
             throw new ScopeException('Unable to receive active session, invalid request scope', $e->getCode(), $e);
130 133
         }
131 134
     }
Please login to merge, or discard this patch.
src/Cookies/CookieManager.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
         string $domain = null,
119 119
         bool $secure = null,
120 120
         bool $httpOnly = true
121
-    ) {
121
+    ){
122 122
         $this->getCookieQueue()->set($name, $value, $lifetime, $path, $domain, $secure, $httpOnly);
123 123
 
124 124
         return $this;
@@ -169,9 +169,9 @@  discard block
 block discarded – undo
169 169
      */
170 170
     private function getRequest(): ServerRequestInterface
171 171
     {
172
-        try {
172
+        try{
173 173
             return $this->container->get(ServerRequestInterface::class);
174
-        } catch (NotFoundExceptionInterface $e) {
174
+        }catch (NotFoundExceptionInterface $e){
175 175
             throw new ScopeException('Unable to receive active request', $e->getCode(), $e);
176 176
         }
177 177
     }
@@ -183,9 +183,9 @@  discard block
 block discarded – undo
183 183
      */
184 184
     private function getCookieQueue(): CookieQueue
185 185
     {
186
-        try {
186
+        try{
187 187
             return $this->container->get(CookieQueue::class);
188
-        } catch (NotFoundExceptionInterface $e) {
188
+        }catch (NotFoundExceptionInterface $e){
189 189
             throw new ScopeException('Unable to receive cookie queue, invalid request scope', $e->getCode(), $e);
190 190
         }
191 191
     }
Please login to merge, or discard this patch.
Braces   +10 added lines, -4 removed lines patch added patch discarded remove patch
@@ -169,9 +169,12 @@  discard block
 block discarded – undo
169 169
      */
170 170
     private function getRequest(): ServerRequestInterface
171 171
     {
172
-        try {
172
+        try
173
+        {
173 174
             return $this->container->get(ServerRequestInterface::class);
174
-        } catch (NotFoundExceptionInterface $e) {
175
+        }
176
+        catch (NotFoundExceptionInterface $e)
177
+        {
175 178
             throw new ScopeException('Unable to receive active request', $e->getCode(), $e);
176 179
         }
177 180
     }
@@ -183,9 +186,12 @@  discard block
 block discarded – undo
183 186
      */
184 187
     private function getCookieQueue(): CookieQueue
185 188
     {
186
-        try {
189
+        try
190
+        {
187 191
             return $this->container->get(CookieQueue::class);
188
-        } catch (NotFoundExceptionInterface $e) {
192
+        }
193
+        catch (NotFoundExceptionInterface $e)
194
+        {
189 195
             throw new ScopeException('Unable to receive cookie queue, invalid request scope', $e->getCode(), $e);
190 196
         }
191 197
     }
Please login to merge, or discard this patch.
src/Auth/AuthScope.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -94,9 +94,9 @@
 block discarded – undo
94 94
      */
95 95
     private function getAuthContext(): AuthContextInterface
96 96
     {
97
-        try {
97
+        try{
98 98
             return $this->container->get(AuthContextInterface::class);
99
-        } catch (NotFoundExceptionInterface $e) {
99
+        }catch (NotFoundExceptionInterface $e){
100 100
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
101 101
         }
102 102
     }
Please login to merge, or discard this patch.
Braces   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -94,9 +94,12 @@
 block discarded – undo
94 94
      */
95 95
     private function getAuthContext(): AuthContextInterface
96 96
     {
97
-        try {
97
+        try
98
+        {
98 99
             return $this->container->get(AuthContextInterface::class);
99
-        } catch (NotFoundExceptionInterface $e) {
100
+        }
101
+        catch (NotFoundExceptionInterface $e)
102
+        {
100 103
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
101 104
         }
102 105
     }
Please login to merge, or discard this patch.
src/Security/GuardScope.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
         PermissionsInterface $permissions,
42 42
         ContainerInterface $actorScope,
43 43
         array $roles = []
44
-    ) {
44
+    ){
45 45
         $this->roles = $roles;
46 46
         $this->container = $actorScope;
47 47
         $this->permissions = $permissions;
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
     public function allows(string $permission, array $context = []): bool
56 56
     {
57 57
         $allows = false;
58
-        foreach ($this->getRoles() as $role) {
59
-            if (!$this->permissions->hasRole($role)) {
58
+        foreach ($this->getRoles() as $role){
59
+            if (!$this->permissions->hasRole($role)){
60 60
                 continue;
61 61
             }
62 62
 
@@ -103,13 +103,13 @@  discard block
 block discarded – undo
103 103
      */
104 104
     public function getActor(): ActorInterface
105 105
     {
106
-        if (!is_null($this->actor)) {
106
+        if (!is_null($this->actor)){
107 107
             return $this->actor;
108 108
         }
109 109
 
110
-        try {
110
+        try{
111 111
             return $this->container->get(ActorInterface::class);
112
-        } catch (NotFoundExceptionInterface $e) {
112
+        }catch (NotFoundExceptionInterface $e){
113 113
             throw new ScopeException('Unable to resolve Actor from the scope', $e->getCode(), $e);
114 114
         }
115 115
     }
Please login to merge, or discard this patch.
Braces   +11 added lines, -5 removed lines patch added patch discarded remove patch
@@ -55,8 +55,10 @@  discard block
 block discarded – undo
55 55
     public function allows(string $permission, array $context = []): bool
56 56
     {
57 57
         $allows = false;
58
-        foreach ($this->getRoles() as $role) {
59
-            if (!$this->permissions->hasRole($role)) {
58
+        foreach ($this->getRoles() as $role)
59
+        {
60
+            if (!$this->permissions->hasRole($role))
61
+            {
60 62
                 continue;
61 63
             }
62 64
 
@@ -103,13 +105,17 @@  discard block
 block discarded – undo
103 105
      */
104 106
     public function getActor(): ActorInterface
105 107
     {
106
-        if (!is_null($this->actor)) {
108
+        if (!is_null($this->actor))
109
+        {
107 110
             return $this->actor;
108 111
         }
109 112
 
110
-        try {
113
+        try
114
+        {
111 115
             return $this->container->get(ActorInterface::class);
112
-        } catch (NotFoundExceptionInterface $e) {
116
+        }
117
+        catch (NotFoundExceptionInterface $e)
118
+        {
113 119
             throw new ScopeException('Unable to resolve Actor from the scope', $e->getCode(), $e);
114 120
         }
115 121
     }
Please login to merge, or discard this patch.
src/Bootloader/Auth/AuthBootloader.php 2 patches
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -51,8 +51,8 @@  discard block
 block discarded – undo
51 51
      */
52 52
     public function getActor(TokenInterface $token): ?object
53 53
     {
54
-        foreach ($this->getProviders() as $provider) {
55
-            if (!$provider instanceof ActorProviderInterface) {
54
+        foreach ($this->getProviders() as $provider){
55
+            if (!$provider instanceof ActorProviderInterface){
56 56
                 throw new AuthException(sprintf(
57 57
                     'Expected `ActorProviderInterface`, got `%s`',
58 58
                     get_class($provider)
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
             }
61 61
 
62 62
             $actor = $provider->getActor($token);
63
-            if ($actor !== null) {
63
+            if ($actor !== null){
64 64
                 return $actor;
65 65
             }
66 66
         }
@@ -83,13 +83,13 @@  discard block
 block discarded – undo
83 83
      */
84 84
     private function getProviders(): \Generator
85 85
     {
86
-        foreach ($this->actorProvider as $provider) {
87
-            if ($provider instanceof Autowire) {
86
+        foreach ($this->actorProvider as $provider){
87
+            if ($provider instanceof Autowire){
88 88
                 yield $provider->resolve($this->factory);
89 89
                 continue;
90 90
             }
91 91
 
92
-            if (is_object($provider)) {
92
+            if (is_object($provider)){
93 93
                 yield $provider;
94 94
                 continue;
95 95
             }
Please login to merge, or discard this patch.
Braces   +12 added lines, -6 removed lines patch added patch discarded remove patch
@@ -51,8 +51,10 @@  discard block
 block discarded – undo
51 51
      */
52 52
     public function getActor(TokenInterface $token): ?object
53 53
     {
54
-        foreach ($this->getProviders() as $provider) {
55
-            if (!$provider instanceof ActorProviderInterface) {
54
+        foreach ($this->getProviders() as $provider)
55
+        {
56
+            if (!$provider instanceof ActorProviderInterface)
57
+            {
56 58
                 throw new AuthException(sprintf(
57 59
                     'Expected `ActorProviderInterface`, got `%s`',
58 60
                     get_class($provider)
@@ -60,7 +62,8 @@  discard block
 block discarded – undo
60 62
             }
61 63
 
62 64
             $actor = $provider->getActor($token);
63
-            if ($actor !== null) {
65
+            if ($actor !== null)
66
+            {
64 67
                 return $actor;
65 68
             }
66 69
         }
@@ -83,13 +86,16 @@  discard block
 block discarded – undo
83 86
      */
84 87
     private function getProviders(): \Generator
85 88
     {
86
-        foreach ($this->actorProvider as $provider) {
87
-            if ($provider instanceof Autowire) {
89
+        foreach ($this->actorProvider as $provider)
90
+        {
91
+            if ($provider instanceof Autowire)
92
+            {
88 93
                 yield $provider->resolve($this->factory);
89 94
                 continue;
90 95
             }
91 96
 
92
-            if (is_object($provider)) {
97
+            if (is_object($provider))
98
+            {
93 99
                 yield $provider;
94 100
                 continue;
95 101
             }
Please login to merge, or discard this patch.
src/Bootloader/Auth/HttpAuthBootloader.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -87,8 +87,8 @@
 block discarded – undo
87 87
         $registry = new TransportRegistry();
88 88
         $registry->setDefaultTransport($config->getDefaultTransport());
89 89
 
90
-        foreach ($config->getTransports() as $name => $transport) {
91
-            if ($transport instanceof Autowire) {
90
+        foreach ($config->getTransports() as $name => $transport){
91
+            if ($transport instanceof Autowire){
92 92
                 $transport = $transport->resolve($factory);
93 93
             }
94 94
 
Please login to merge, or discard this patch.
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -87,8 +87,10 @@
 block discarded – undo
87 87
         $registry = new TransportRegistry();
88 88
         $registry->setDefaultTransport($config->getDefaultTransport());
89 89
 
90
-        foreach ($config->getTransports() as $name => $transport) {
91
-            if ($transport instanceof Autowire) {
90
+        foreach ($config->getTransports() as $name => $transport)
91
+        {
92
+            if ($transport instanceof Autowire)
93
+            {
92 94
                 $transport = $transport->resolve($factory);
93 95
             }
94 96
 
Please login to merge, or discard this patch.
src/Bootloader/Auth/SecurityActorBootloader.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
     private function actor(AuthContextInterface $context): ActorInterface
39 39
     {
40 40
         $actor = $context->getActor();
41
-        if ($actor instanceof ActorInterface) {
41
+        if ($actor instanceof ActorInterface){
42 42
             return $actor;
43 43
         }
44 44
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,8 @@
 block discarded – undo
38 38
     private function actor(AuthContextInterface $context): ActorInterface
39 39
     {
40 40
         $actor = $context->getActor();
41
-        if ($actor instanceof ActorInterface) {
41
+        if ($actor instanceof ActorInterface)
42
+        {
42 43
             return $actor;
43 44
         }
44 45
 
Please login to merge, or discard this patch.
src/Auth/Config/AuthConfig.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
     public function getTransports(): array
42 42
     {
43 43
         $transports = [];
44
-        foreach ($this->config['transports'] as $name => $transport) {
45
-            if (is_object($transport) && !$transport instanceof Autowire) {
44
+        foreach ($this->config['transports'] as $name => $transport){
45
+            if (is_object($transport) && !$transport instanceof Autowire){
46 46
                 $transports[$name] = $transport;
47 47
                 continue;
48 48
             }
@@ -61,15 +61,15 @@  discard block
 block discarded – undo
61 61
      */
62 62
     private function wire($item): Autowire
63 63
     {
64
-        if ($item instanceof Autowire) {
64
+        if ($item instanceof Autowire){
65 65
             return $item;
66 66
         }
67 67
 
68
-        if (is_string($item)) {
68
+        if (is_string($item)){
69 69
             return new Autowire($item);
70 70
         }
71 71
 
72
-        if (is_array($item) && isset($item['class'])) {
72
+        if (is_array($item) && isset($item['class'])){
73 73
             return new Autowire($item['class'], $item['options'] ?? []);
74 74
         }
75 75
 
Please login to merge, or discard this patch.
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -41,8 +41,10 @@  discard block
 block discarded – undo
41 41
     public function getTransports(): array
42 42
     {
43 43
         $transports = [];
44
-        foreach ($this->config['transports'] as $name => $transport) {
45
-            if (is_object($transport) && !$transport instanceof Autowire) {
44
+        foreach ($this->config['transports'] as $name => $transport)
45
+        {
46
+            if (is_object($transport) && !$transport instanceof Autowire)
47
+            {
46 48
                 $transports[$name] = $transport;
47 49
                 continue;
48 50
             }
@@ -61,15 +63,18 @@  discard block
 block discarded – undo
61 63
      */
62 64
     private function wire($item): Autowire
63 65
     {
64
-        if ($item instanceof Autowire) {
66
+        if ($item instanceof Autowire)
67
+        {
65 68
             return $item;
66 69
         }
67 70
 
68
-        if (is_string($item)) {
71
+        if (is_string($item))
72
+        {
69 73
             return new Autowire($item);
70 74
         }
71 75
 
72
-        if (is_array($item) && isset($item['class'])) {
76
+        if (is_array($item) && isset($item['class']))
77
+        {
73 78
             return new Autowire($item['class'], $item['options'] ?? []);
74 79
         }
75 80
 
Please login to merge, or discard this patch.
src/Auth/Session/Token.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@
 block discarded – undo
100 100
     public static function unpack(array $data): Token
101 101
     {
102 102
         $expiresAt = null;
103
-        if ($data['expiresAt'] != null) {
103
+        if ($data['expiresAt'] != null){
104 104
             $expiresAt = (new \DateTimeImmutable())->setTimestamp($data['expiresAt']);
105 105
         }
106 106
 
Please login to merge, or discard this patch.
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -100,7 +100,8 @@
 block discarded – undo
100 100
     public static function unpack(array $data): Token
101 101
     {
102 102
         $expiresAt = null;
103
-        if ($data['expiresAt'] != null) {
103
+        if ($data['expiresAt'] != null)
104
+        {
104 105
             $expiresAt = (new \DateTimeImmutable())->setTimestamp($data['expiresAt']);
105 106
         }
106 107
 
Please login to merge, or discard this patch.