ARCANESOFT / Foundation

src/Authorization/Policies/PermissionsPolicy.php

Braces +3 added lines, -2 removed lines

@@ -109,7 +109,8 @@
      */
     public function detachRole(Administrator $administrator, Permission $permission = null, Role $role = null)
     {
-        if ( ! is_null($role))
-            return ! $role->isLocked();
+        if ( ! is_null($role)) {
+                    return ! $role->isLocked();
+        }
     }
 }

src/Authorization/Policies/AdministratorsPolicy.php

Braces +33 added lines, -22 removed lines

@@ -147,8 +147,9 @@ 
      */
     public function show(Administrator $administrator, Administrator $model = null)
     {
-        if ($model && $model->isSuperAdmin() && ! $administrator->isSuperAdmin())
-            return false;
+        if ($model && $model->isSuperAdmin() && ! $administrator->isSuperAdmin()) {
+                    return false;
+        }
     }
 
     /**
@@ -186,14 +187,17 @@ 
      */
     public function activate(Administrator $administrator, Administrator $model = null)
     {
-        if ( ! is_null($model) && $model->isActive())
-            return false;
+        if ( ! is_null($model) && $model->isActive()) {
+                    return false;
+        }
 
-        if ($administrator->is($model))
-            return false;
+        if ($administrator->is($model)) {
+                    return false;
+        }
 
-        if ( ! is_null($model) && $model->isSuperAdmin())
-            return false;
+        if ( ! is_null($model) && $model->isSuperAdmin()) {
+                    return false;
+        }
     }
 
     /**
@@ -206,14 +210,17 @@ 
      */
     public function deactivate(Administrator $administrator, Administrator $model = null)
     {
-        if ( ! is_null($model) && ! $model->isActive())
-            return false;
+        if ( ! is_null($model) && ! $model->isActive()) {
+                    return false;
+        }
 
-        if ($administrator->is($model))
-            return false;
+        if ($administrator->is($model)) {
+                    return false;
+        }
 
-        if ( ! is_null($model) && $model->isSuperAdmin())
-            return false;
+        if ( ! is_null($model) && $model->isSuperAdmin()) {
+                    return false;
+        }
     }
 
     /**
@@ -226,11 +233,13 @@ 
      */
     public function delete(Administrator $administrator, Administrator $model = null)
     {
-        if ($administrator->is($model))
-            return false;
+        if ($administrator->is($model)) {
+                    return false;
+        }
 
-        if ( ! is_null($model))
-            return $model->isDeletable();
+        if ( ! is_null($model)) {
+                    return $model->isDeletable();
+        }
     }
 
     /**
@@ -243,8 +252,9 @@ 
      */
     public function forceDelete(Administrator $administrator, Administrator $model = null)
     {
-        if ( ! is_null($model))
-            return $model->isDeletable();
+        if ( ! is_null($model)) {
+                    return $model->isDeletable();
+        }
     }
 
     /**
@@ -257,7 +267,8 @@ 
      */
     public function restore(Administrator $administrator, Administrator $model = null)
     {
-        if ( ! is_null($model))
-            return $model->trashed();
+        if ( ! is_null($model)) {
+                    return $model->trashed();
+        }
     }
 }

src/Authorization/Policies/RolesPolicy.php

Braces +27 added lines, -18 removed lines

@@ -129,8 +129,9 @@ 
      */
     public function show(Administrator $administrator, Role $role = null)
     {
-        if ($role && $role->isAdministrator() && ! $administrator->isSuperAdmin())
-            return false;
+        if ($role && $role->isAdministrator() && ! $administrator->isSuperAdmin()) {
+                    return false;
+        }
     }
 
     /**
@@ -155,8 +156,9 @@ 
      */
     public function update(Administrator $administrator, Role $role = null)
     {
-        if (static::isRoleLocked($role))
-            return false;
+        if (static::isRoleLocked($role)) {
+                    return false;
+        }
     }
 
     /**
@@ -169,11 +171,13 @@ 
      */
     public function activate(Administrator $administrator, Role $role = null)
     {
-        if (static::isRoleLocked($role))
-            return false;
+        if (static::isRoleLocked($role)) {
+                    return false;
+        }
 
-        if ( ! is_null($role) && $role->isActive())
-            return false;
+        if ( ! is_null($role) && $role->isActive()) {
+                    return false;
+        }
     }
 
     /**
@@ -186,11 +190,13 @@ 
      */
     public function deactivate(Administrator $administrator, Role $role = null)
     {
-        if (static::isRoleLocked($role))
-            return false;
+        if (static::isRoleLocked($role)) {
+                    return false;
+        }
 
-        if ( ! is_null($role) && ! $role->isActive())
-            return false;
+        if ( ! is_null($role) && ! $role->isActive()) {
+                    return false;
+        }
     }
 
     /**
@@ -203,8 +209,9 @@ 
      */
     public function delete(Administrator $administrator, Role $role = null)
     {
-        if ( ! is_null($role))
-            return $role->isDeletable();
+        if ( ! is_null($role)) {
+                    return $role->isDeletable();
+        }
     }
 
     /**
@@ -218,8 +225,9 @@ 
      */
     public function detachAdministrator(Administrator $administrator, Role $role = null, Administrator $model = null)
     {
-        if ( ! $administrator->isSuperAdmin() && $model->isSuperAdmin())
-            return false;
+        if ( ! $administrator->isSuperAdmin() && $model->isSuperAdmin()) {
+                    return false;
+        }
     }
 
     /**
@@ -233,8 +241,9 @@ 
      */
     public function detachPermission(Administrator $administrator, Role $role = null, Permission $model = null)
     {
-        if (static::isRoleLocked($role))
-            return false;
+        if (static::isRoleLocked($role)) {
+                    return false;
+        }
     }
 
     /* -----------------------------------------------------------------

src/Authorization/Models/Concerns/HasPassword.php

Braces +3 added lines, -2 removed lines

@@ -28,8 +28,9 @@
      */
     public function setPasswordAttribute($password)
     {
-        if (is_null($password))
-            return;
+        if (is_null($password)) {
+                    return;
+        }
 
         $this->attributes['password'] = Hash::make($password);
     }

src/Authorization/Models/Presenters/AuthenticatablePresenter.php

Braces +12 added lines, -8 removed lines

@@ -34,8 +34,9 @@ 
      */
     public function getDisplayNameAttribute(): string
     {
-        if ($this->hasName())
-            return $this->getFullNameAttribute();
+        if ($this->hasName()) {
+                    return $this->getFullNameAttribute();
+        }
 
         return $this->username;
     }
@@ -57,8 +58,9 @@ 
      */
     public function setFirstNameAttribute($firstName)
     {
-        if (is_null($firstName))
-            return;
+        if (is_null($firstName)) {
+                    return;
+        }
 
         $this->attributes['first_name'] = Str::title(Str::lower($firstName));
     }
@@ -70,8 +72,9 @@ 
      */
     public function setLastNameAttribute($lastName)
     {
-        if (is_null($lastName))
-            return;
+        if (is_null($lastName)) {
+                    return;
+        }
 
         $this->attributes['last_name'] = Str::upper($lastName);
     }
@@ -135,8 +138,9 @@ 
      */
     public function getLastActivityAttribute(): string
     {
-        if (is_null($this->last_activity_at))
-            return __('No recent activity');
+        if (is_null($this->last_activity_at)) {
+                    return __('No recent activity');
+        }
 
         return $this->last_activity_at->diffForHumans();
     }

src/Authorization/Models/PermissionsGroup.php

Braces +18 added lines, -12 removed lines

@@ -170,8 +170,9 @@ 
      */
     public function attachPermission(&$permission, bool $reload = true)
     {
-        if ($this->hasPermission($permission))
-            return;
+        if ($this->hasPermission($permission)) {
+                    return;
+        }
 
         event(new AttachingPermission($this, $permission));
         $permission = $this->permissions()->save($permission);
@@ -192,8 +193,9 @@ 
     {
         $permission = $this->getPermissionById($id);
 
-        if ($permission !== null)
-            $this->attachPermission($permission, $reload);
+        if ($permission !== null) {
+                    $this->attachPermission($permission, $reload);
+        }
 
         return $permission;
     }
@@ -225,8 +227,9 @@ 
      */
     public function detachPermission(&$permission, bool $reload = true)
     {
-        if ( ! $this->hasPermission($permission))
-            return;
+        if ( ! $this->hasPermission($permission)) {
+                    return;
+        }
 
         $permission = $this->getPermissionFromGroup($permission);
 
@@ -247,8 +250,9 @@ 
      */
     public function detachPermissionById($id, bool $reload = true)
     {
-        if ( ! is_null($permission = $this->getPermissionById($id)))
-            $this->detachPermission($permission, $reload);
+        if ( ! is_null($permission = $this->getPermissionById($id))) {
+                    $this->detachPermission($permission, $reload);
+        }
 
         return $permission;
     }
@@ -296,8 +300,9 @@ 
      */
     public function hasPermission($id)
     {
-        if ($id instanceof Model)
-            $id = $id->getKey();
+        if ($id instanceof Model) {
+                    $id = $id->getKey();
+        }
 
         return $this->getPermissionFromGroup($id) !== null;
     }
@@ -316,8 +321,9 @@ 
      */
     private function getPermissionFromGroup($id)
     {
-        if ($id instanceof Model)
-            $id = $id->getKey();
+        if ($id instanceof Model) {
+                    $id = $id->getKey();
+        }
 
         $this->loadPermissions();
 

src/Authorization/Models/Role.php

Braces +9 added lines, -6 removed lines

@@ -273,8 +273,9 @@ 
      */
     public function attachUser($administrator, bool $reload = true): void
     {
-        if ($this->hasAdministrator($administrator))
-            return;
+        if ($this->hasAdministrator($administrator)) {
+                    return;
+        }
 
         event(new AttachingAdministrator($this, $administrator));
         $this->administrators()->attach($administrator);
@@ -314,8 +315,9 @@ 
      */
     public function attachPermission($permission, bool $reload = true): void
     {
-        if ($this->hasPermission($permission))
-            return;
+        if ($this->hasPermission($permission)) {
+                    return;
+        }
 
         event(new AttachingPermission($this, $permission));
         $this->permissions()->attach($permission);
@@ -366,8 +368,9 @@ 
      */
     public function can(string $ability): bool
     {
-        if ( ! $this->isActive())
-            return false;
+        if ( ! $this->isActive()) {
+                    return false;
+        }
 
         return $this->permissions->filter(function (Permission $permission) use ($ability) {
             return $permission->hasAbility($ability);

src/Authorization/Models/Administrator.php

Braces +6 added lines, -4 removed lines

@@ -287,8 +287,9 @@ 
      */
     public function canImpersonate(): bool
     {
-        if ( ! impersonator()->isEnabled())
-            return false;
+        if ( ! impersonator()->isEnabled()) {
+                    return false;
+        }
 
         return $this->isSuperAdmin();
     }
@@ -300,8 +301,9 @@ 
      */
     public function canBeImpersonated(): bool
     {
-        if ( ! impersonator()->isEnabled())
-            return false;
+        if ( ! impersonator()->isEnabled()) {
+                    return false;
+        }
 
         return false;
     }

src/Authorization/Models/TwoFactor.php

Braces +3 added lines, -2 removed lines

@@ -151,8 +151,9 @@
     public function getValidRecoveryCode(string $code)
     {
         foreach ($this->decrypted_recovery_codes as $recoveryCode) {
-            if (hash_equals($code, $recoveryCode))
-                return $recoveryCode;
+            if (hash_equals($code, $recoveryCode)) {
+                            return $recoveryCode;
+            }
         }
 
         return null;