Passed
Pull Request — master (#14)
by ARCANEDEV
05:47
created
src/Authorization/Models/Concerns/HasPassword.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -28,8 +28,9 @@
 block discarded – undo
28 28
      */
29 29
     public function setPasswordAttribute($password)
30 30
     {
31
-        if (is_null($password))
32
-            return;
31
+        if (is_null($password)) {
32
+                    return;
33
+        }
33 34
 
34 35
         $this->attributes['password'] = Hash::make($password);
35 36
     }
Please login to merge, or discard this patch.
src/Authorization/Models/Presenters/PermissionPresenter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
      */
31 31
     public function getUsersCountAttribute(): int
32 32
     {
33
-        return $this->roles->sum(function (Role $role) {
33
+        return $this->roles->sum(function(Role $role) {
34 34
             return $role->administrators->count();
35 35
         });
36 36
     }
Please login to merge, or discard this patch.
src/Authorization/Models/Presenters/AuthenticatablePresenter.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@
 block discarded – undo
96 96
         $show  = 2;
97 97
         $parts = explode('@', $this->email);
98 98
 
99
-        return substr($parts[0],0, $show).str_repeat('*', 3).'@'.$parts[1];
99
+        return substr($parts[0], 0, $show).str_repeat('*', 3).'@'.$parts[1];
100 100
     }
101 101
 
102 102
     /**
Please login to merge, or discard this patch.
Braces   +12 added lines, -8 removed lines patch added patch discarded remove patch
@@ -34,8 +34,9 @@  discard block
 block discarded – undo
34 34
      */
35 35
     public function getDisplayNameAttribute(): string
36 36
     {
37
-        if ($this->hasName())
38
-            return $this->getFullNameAttribute();
37
+        if ($this->hasName()) {
38
+                    return $this->getFullNameAttribute();
39
+        }
39 40
 
40 41
         return $this->username;
41 42
     }
@@ -57,8 +58,9 @@  discard block
 block discarded – undo
57 58
      */
58 59
     public function setFirstNameAttribute($firstName)
59 60
     {
60
-        if (is_null($firstName))
61
-            return;
61
+        if (is_null($firstName)) {
62
+                    return;
63
+        }
62 64
 
63 65
         $this->attributes['first_name'] = Str::title(Str::lower($firstName));
64 66
     }
@@ -70,8 +72,9 @@  discard block
 block discarded – undo
70 72
      */
71 73
     public function setLastNameAttribute($lastName)
72 74
     {
73
-        if (is_null($lastName))
74
-            return;
75
+        if (is_null($lastName)) {
76
+                    return;
77
+        }
75 78
 
76 79
         $this->attributes['last_name'] = Str::upper($lastName);
77 80
     }
@@ -135,8 +138,9 @@  discard block
 block discarded – undo
135 138
      */
136 139
     public function getLastActivityAttribute(): string
137 140
     {
138
-        if (is_null($this->last_activity_at))
139
-            return __('No recent activity');
141
+        if (is_null($this->last_activity_at)) {
142
+                    return __('No recent activity');
143
+        }
140 144
 
141 145
         return $this->last_activity_at->diffForHumans();
142 146
     }
Please login to merge, or discard this patch.
src/Authorization/Models/PermissionsGroup.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -311,7 +311,7 @@
 block discarded – undo
311 311
         $this->loadPermissions();
312 312
 
313 313
         return $this->permissions
314
-            ->filter(function (Permission $permission) use ($id) {
314
+            ->filter(function(Permission $permission) use ($id) {
315 315
                 return $permission->getKey() == $id;
316 316
             })
317 317
             ->first();
Please login to merge, or discard this patch.
Braces   +18 added lines, -12 removed lines patch added patch discarded remove patch
@@ -170,8 +170,9 @@  discard block
 block discarded – undo
170 170
      */
171 171
     public function attachPermission(&$permission, bool $reload = true)
172 172
     {
173
-        if ($this->hasPermission($permission))
174
-            return;
173
+        if ($this->hasPermission($permission)) {
174
+                    return;
175
+        }
175 176
 
176 177
         event(new AttachingPermission($this, $permission));
177 178
         $permission = $this->permissions()->save($permission);
@@ -192,8 +193,9 @@  discard block
 block discarded – undo
192 193
     {
193 194
         $permission = $this->getPermissionById($id);
194 195
 
195
-        if ($permission !== null)
196
-            $this->attachPermission($permission, $reload);
196
+        if ($permission !== null) {
197
+                    $this->attachPermission($permission, $reload);
198
+        }
197 199
 
198 200
         return $permission;
199 201
     }
@@ -225,8 +227,9 @@  discard block
 block discarded – undo
225 227
      */
226 228
     public function detachPermission(&$permission, bool $reload = true)
227 229
     {
228
-        if ( ! $this->hasPermission($permission))
229
-            return;
230
+        if ( ! $this->hasPermission($permission)) {
231
+                    return;
232
+        }
230 233
 
231 234
         $permission = $this->getPermissionFromGroup($permission);
232 235
 
@@ -247,8 +250,9 @@  discard block
 block discarded – undo
247 250
      */
248 251
     public function detachPermissionById($id, bool $reload = true)
249 252
     {
250
-        if ( ! is_null($permission = $this->getPermissionById($id)))
251
-            $this->detachPermission($permission, $reload);
253
+        if ( ! is_null($permission = $this->getPermissionById($id))) {
254
+                    $this->detachPermission($permission, $reload);
255
+        }
252 256
 
253 257
         return $permission;
254 258
     }
@@ -296,8 +300,9 @@  discard block
 block discarded – undo
296 300
      */
297 301
     public function hasPermission($id)
298 302
     {
299
-        if ($id instanceof Model)
300
-            $id = $id->getKey();
303
+        if ($id instanceof Model) {
304
+                    $id = $id->getKey();
305
+        }
301 306
 
302 307
         return $this->getPermissionFromGroup($id) !== null;
303 308
     }
@@ -316,8 +321,9 @@  discard block
 block discarded – undo
316 321
      */
317 322
     private function getPermissionFromGroup($id)
318 323
     {
319
-        if ($id instanceof Model)
320
-            $id = $id->getKey();
324
+        if ($id instanceof Model) {
325
+                    $id = $id->getKey();
326
+        }
321 327
 
322 328
         $this->loadPermissions();
323 329
 
Please login to merge, or discard this patch.
src/Authorization/Models/Role.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
      */
166 166
     public function scopeFilterByAuthenticatedAdministrator(Builder $query, Administrator $administrator): Builder
167 167
     {
168
-        return $query->unless($administrator->isSuperAdmin(), function (Builder $q) {
168
+        return $query->unless($administrator->isSuperAdmin(), function(Builder $q) {
169 169
             $q->where('key', '!=', static::ADMINISTRATOR);
170 170
         });
171 171
     }
@@ -369,7 +369,7 @@  discard block
 block discarded – undo
369 369
         if ( ! $this->isActive())
370 370
             return false;
371 371
 
372
-        return $this->permissions->filter(function (Permission $permission) use ($ability) {
372
+        return $this->permissions->filter(function(Permission $permission) use ($ability) {
373 373
             return $permission->hasAbility($ability);
374 374
         })->first() !== null;
375 375
     }
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
386 386
     {
387 387
         $permissions = is_array($permissions) ? collect($permissions) : $permissions;
388 388
 
389
-        $failed = $permissions->reject(function ($permission) {
389
+        $failed = $permissions->reject(function($permission) {
390 390
             return $this->can($permission);
391 391
         })->values();
392 392
 
Please login to merge, or discard this patch.
Braces   +9 added lines, -6 removed lines patch added patch discarded remove patch
@@ -273,8 +273,9 @@  discard block
 block discarded – undo
273 273
      */
274 274
     public function attachUser($administrator, bool $reload = true): void
275 275
     {
276
-        if ($this->hasAdministrator($administrator))
277
-            return;
276
+        if ($this->hasAdministrator($administrator)) {
277
+                    return;
278
+        }
278 279
 
279 280
         event(new AttachingAdministrator($this, $administrator));
280 281
         $this->administrators()->attach($administrator);
@@ -314,8 +315,9 @@  discard block
 block discarded – undo
314 315
      */
315 316
     public function attachPermission($permission, bool $reload = true): void
316 317
     {
317
-        if ($this->hasPermission($permission))
318
-            return;
318
+        if ($this->hasPermission($permission)) {
319
+                    return;
320
+        }
319 321
 
320 322
         event(new AttachingPermission($this, $permission));
321 323
         $this->permissions()->attach($permission);
@@ -366,8 +368,9 @@  discard block
 block discarded – undo
366 368
      */
367 369
     public function can(string $ability): bool
368 370
     {
369
-        if ( ! $this->isActive())
370
-            return false;
371
+        if ( ! $this->isActive()) {
372
+                    return false;
373
+        }
371 374
 
372 375
         return $this->permissions->filter(function (Permission $permission) use ($ability) {
373 376
             return $permission->hasAbility($ability);
Please login to merge, or discard this patch.
src/Authorization/Models/Administrator.php 2 patches
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
         return $this->active_roles
166 166
             ->pluck('permissions')
167 167
             ->flatten()
168
-            ->unique(function (Permission $permission) {
168
+            ->unique(function(Permission $permission) {
169 169
                 return $permission->getKey();
170 170
             });
171 171
     }
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
      */
210 210
     public function may($ability): bool
211 211
     {
212
-        return $this->permissions->filter(function (Permission $permission) use ($ability) {
212
+        return $this->permissions->filter(function(Permission $permission) use ($ability) {
213 213
             return $permission->hasAbility($ability);
214 214
         })->isNotEmpty();
215 215
     }
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
             ? $permissions
229 229
             : $this->newCollection($permissions);
230 230
 
231
-        $failed = $permissions->reject(function ($permission) {
231
+        $failed = $permissions->reject(function($permission) {
232 232
             return $this->may($permission);
233 233
         })->values();
234 234
 
Please login to merge, or discard this patch.
Braces   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -287,8 +287,9 @@  discard block
 block discarded – undo
287 287
      */
288 288
     public function canImpersonate(): bool
289 289
     {
290
-        if ( ! impersonator()->isEnabled())
291
-            return false;
290
+        if ( ! impersonator()->isEnabled()) {
291
+                    return false;
292
+        }
292 293
 
293 294
         return $this->isSuperAdmin();
294 295
     }
@@ -300,8 +301,9 @@  discard block
 block discarded – undo
300 301
      */
301 302
     public function canBeImpersonated(): bool
302 303
     {
303
-        if ( ! impersonator()->isEnabled())
304
-            return false;
304
+        if ( ! impersonator()->isEnabled()) {
305
+                    return false;
306
+        }
305 307
 
306 308
         return false;
307 309
     }
Please login to merge, or discard this patch.
src/Authorization/Models/TwoFactor.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -151,8 +151,9 @@
 block discarded – undo
151 151
     public function getValidRecoveryCode(string $code)
152 152
     {
153 153
         foreach ($this->decrypted_recovery_codes as $recoveryCode) {
154
-            if (hash_equals($code, $recoveryCode))
155
-                return $recoveryCode;
154
+            if (hash_equals($code, $recoveryCode)) {
155
+                            return $recoveryCode;
156
+            }
156 157
         }
157 158
 
158 159
         return null;
Please login to merge, or discard this patch.
src/Authorization/Providers/SessionServiceProvider.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
      */
44 44
     protected function registerCustomSessionHandler(SessionManager $session): void
45 45
     {
46
-        $session->extend('arcanesoft', function (Container $app) {
46
+        $session->extend('arcanesoft', function(Container $app) {
47 47
             return new DatabaseSessionHandler($app->make(SessionsRepository::class), $app);
48 48
         });
49 49
     }
Please login to merge, or discard this patch.
src/Authorization/Auth.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -169,8 +169,9 @@
 block discarded – undo
169 169
     {
170 170
         $emails = (array) static::config('administrators.emails');
171 171
 
172
-        if (empty($emails))
173
-            return false;
172
+        if (empty($emails)) {
173
+                    return false;
174
+        }
174 175
 
175 176
         return in_array($user->email, $emails);
176 177
     }
Please login to merge, or discard this patch.