ARCANESOFT / Foundation

src/Authorization/Console/MakeAdmin.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
      */
     protected static function defaultCreateUserCallback(): Closure
     {
-        return function (string $firstName, string $lastName, string $email, string $password) {
+        return function(string $firstName, string $lastName, string $email, string $password) {
             $repo = static::getAdministratorsRepository();
 
             $administrator = $repo->createOne([

src/Authorization/Console/MakeUser.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
      */
     protected static function defaultCreateUserCallback(): Closure
     {
-        return function (string $firstName, string $lastName, string $email, string $password) {
+        return function(string $firstName, string $lastName, string $email, string $password) {
             /** @var  \Arcanesoft\Foundation\Authorization\Repositories\UsersRepository  $repo */
             $repo = app(UsersRepository::class);
             $now  = now();

src/Authorization/Policies/UsersPolicy.php

Braces +12 added lines, -8 removed lines

@@ -197,8 +197,9 @@ 
      */
     public function activate(Administrator $administrator, User $user = null)
     {
-        if ( ! is_null($user) && $user->isActive())
-            return false;
+        if ( ! is_null($user) && $user->isActive()) {
+                    return false;
+        }
     }
 
     /**
@@ -211,8 +212,9 @@ 
      */
     public function deactivate(Administrator $administrator, User $user = null)
     {
-        if ( ! is_null($user) && ! $user->isActive())
-            return false;
+        if ( ! is_null($user) && ! $user->isActive()) {
+                    return false;
+        }
     }
 
     /**
@@ -225,8 +227,9 @@ 
      */
     public function delete(Administrator $administrator, User $user = null)
     {
-        if ( ! is_null($user))
-            return $user->isDeletable();
+        if ( ! is_null($user)) {
+                    return $user->isDeletable();
+        }
     }
 
     /**
@@ -239,8 +242,9 @@ 
      */
     public function forceDelete(Administrator $administrator, User $user = null)
     {
-        if ( ! is_null($user))
-            return $user->isDeletable();
+        if ( ! is_null($user)) {
+                    return $user->isDeletable();
+        }
     }
 
     /**

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/HasRoles.php

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
      */
     public function getActiveRolesAttribute()
     {
-        return $this->roles->filter(function (Role $role) {
+        return $this->roles->filter(function(Role $role) {
             return $role->isActive();
         });
     }
@@ -93,7 +93,7 @@ 
     {
         $roles = is_array($roles) ? new Collection($roles) : $roles;
 
-        $failed = $roles->reject(function ($role) {
+        $failed = $roles->reject(function($role) {
             return $this->hasRoleKey($role);
         })->values();
 
@@ -109,7 +109,7 @@ 
      */
     public function hasRoleKey(string $key): bool
     {
-        return $this->active_roles->filter(function (Role $role) use ($key) {
+        return $this->active_roles->filter(function(Role $role) use ($key) {
             return $role->hasKey($key);
         })->isNotEmpty();
     }

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/PermissionPresenter.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
      */
     public function getUsersCountAttribute(): int
     {
-        return $this->roles->sum(function (Role $role) {
+        return $this->roles->sum(function(Role $role) {
             return $role->administrators->count();
         });
     }