z1haze / laravel-acl

src/z1haze/Acl/Traits/UserAndLevel.php

Doc Comments +1 added lines, -1 removed lines

@@ -354,7 +354,7 @@
      * Modify a given set of permissions, given the action to be taken
      *
      * @param $permissions
-     * @param $action
+     * @param string $action
      */
     protected function modifyPermissions($permissions, $action)
     {

Unused Use Statements +1 added lines, -1 removed lines

@@ -3,9 +3,9 @@
 namespace z1haze\Acl\Traits;
 
 use Illuminate\Support\Collection;
+use Illuminate\Support\Facades\Cache;
 use ReflectionClass;
 use z1haze\Acl\Exceptions\PermissionNotFoundException;
-use Illuminate\Support\Facades\Cache;
 use z1haze\Acl\Models\Level;
 use z1haze\Acl\Models\Permission;
 

Spacing +11 added lines, -13 removed lines

@@ -122,7 +122,7 @@ 
         return Cache::remember(
             'laravel-acl.getPermissionsFor' . (new ReflectionClass($this))->getShortName() . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cachegetPermissions();
             }
         );
@@ -142,7 +142,7 @@ 
         return Cache::remember(
             'laravel-acl.getAllPermissionsFor' . (new ReflectionClass($this))->getShortName() . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetAllPermissions();
             }
         );
@@ -162,7 +162,7 @@ 
         return Cache::remember(
             'laravel-acl.getInheritedPermissionsFor' . (new ReflectionClass($this))->getShortName() . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetInheritedPermissions();
             }
         );
@@ -182,7 +182,7 @@ 
         return Cache::remember(
             'laravel-acl.getAvailablePermissionsFor' . (new ReflectionClass($this))->getShortName() . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetAvailablePermissions();
             }
         );
@@ -203,7 +203,7 @@ 
         return Cache::remember(
             'laravel-acl.hasPermissionTo_' . (new ReflectionClass($this))->getShortName() . '_' . $this->id . '_Permission_' . $permission,
             config('laravel-acl.cacheMinutes'),
-            function () use ($permission) {
+            function() use ($permission) {
                 return $this->cacheHasPermissionTo($permission);
             }
         );
@@ -281,7 +281,7 @@ 
 
         $allPerms = new Collection();
 
-        $levels->each(function ($level, $key) use (&$allPerms) {
+        $levels->each(function($level, $key) use (&$allPerms) {
             $allPerms = $allPerms->merge($level->permissions);
         });
 
@@ -300,12 +300,11 @@ 
         $rank = is_a($this, config('laravel-acl.level'), true) ? $this->rank : $this->level->rank;
 
         $levels = is_a($this, config('laravel-acl.user'), true) ?
-            config('laravel-acl.level', Level::class)::where('rank', '>=', $rank)->with('permissions')->get() :
-            config('laravel-acl.level', Level::class)::where('rank', '>', $rank)->with('permissions')->get();
+            config('laravel-acl.level', Level::class)::where('rank', '>=', $rank)->with('permissions')->get() : config('laravel-acl.level', Level::class)::where('rank', '>', $rank)->with('permissions')->get();
 
         $inheritedPerms = new Collection();
 
-        $levels->each(function ($level, $key) use (&$inheritedPerms) {
+        $levels->each(function($level, $key) use (&$inheritedPerms) {
             $inheritedPerms = $inheritedPerms->merge($level->permissions->load('level'));
         });
 
@@ -399,8 +398,7 @@ 
                 $permissionsObjects[$permission->id] = ['negated' => true];
             } else {
                 is_a($this, config('laravel-acl.user')) ?
-                    array_push($permissionsObjects, $permission->id) :
-                    array_push($permissionsObjects, $permission);
+                    array_push($permissionsObjects, $permission->id) : array_push($permissionsObjects, $permission);
             }
         }
 
@@ -434,7 +432,7 @@ 
             $this->permissions()->detach($permissionObjects);
 
         if (is_a($this, config('laravel-acl.level'))) {
-            config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) {
+            config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function($permission) {
                 return $permission->id;
             }, $permissionObjects))->update(['level_id' => null]);
         }
@@ -454,7 +452,7 @@ 
         if (is_a($this, config('laravel-acl.level'))) {
             $this->clearPermissions();
 
-            config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) {
+            config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function($permission) {
                 return $permission->id;
             }, $permissionObjects))
                 ->update(['level_id' => $this->id]);

Braces +27 added lines, -18 removed lines

@@ -101,11 +101,13 @@ 
      */
     public function clearPermissions()
     {
-        if (is_a($this, config('laravel-acl.user'), true))
-            $this->permissions()->detach();
+        if (is_a($this, config('laravel-acl.user'), true)) {
+                    $this->permissions()->detach();
+        }
 
-        if (is_a($this, config('laravel-acl.level'), true))
-            config('laravel-acl.permission', Permission::class)::whereLevelId($this->id)->update(['level_id' => null]);
+        if (is_a($this, config('laravel-acl.level'), true)) {
+                    config('laravel-acl.permission', Permission::class)::whereLevelId($this->id)->update(['level_id' => null]);
+        }
     }
 
     /**
@@ -415,11 +417,13 @@ 
      */
     protected function addOrNegate($permissionObjects)
     {
-        if (is_a($this, config('laravel-acl.user')))
-            $this->permissions()->attach($permissionObjects);
+        if (is_a($this, config('laravel-acl.user'))) {
+                    $this->permissions()->attach($permissionObjects);
+        }
 
-        if (is_a($this, config('laravel-acl.level')))
-            $this->permissions()->saveMany($permissionObjects);
+        if (is_a($this, config('laravel-acl.level'))) {
+                    $this->permissions()->saveMany($permissionObjects);
+        }
     }
 
     /**
@@ -430,8 +434,9 @@ 
      */
     protected function remove($permissionObjects)
     {
-        if (is_a($this, config('laravel-acl.user')))
-            $this->permissions()->detach($permissionObjects);
+        if (is_a($this, config('laravel-acl.user'))) {
+                    $this->permissions()->detach($permissionObjects);
+        }
 
         if (is_a($this, config('laravel-acl.level'))) {
             config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) {
@@ -448,8 +453,9 @@ 
      */
     protected function sync($permissionObjects)
     {
-        if (is_a($this, config('laravel-acl.user')))
-            $this->permissions()->sync($permissionObjects);
+        if (is_a($this, config('laravel-acl.user'))) {
+                    $this->permissions()->sync($permissionObjects);
+        }
 
         if (is_a($this, config('laravel-acl.level'))) {
             $this->clearPermissions();
@@ -472,14 +478,17 @@ 
      */
     protected function getPermission($permission)
     {
-        if (is_string($permission))
-            $permission = config('laravel-acl.permission', Permission::class)::whereName($permission)->first();
+        if (is_string($permission)) {
+                    $permission = config('laravel-acl.permission', Permission::class)::whereName($permission)->first();
+        }
 
-        if (is_int($permission))
-            $permission = config('laravel-acl.permission', Permission::class)::find($permission);
+        if (is_int($permission)) {
+                    $permission = config('laravel-acl.permission', Permission::class)::find($permission);
+        }
 
-        if (!$permission)
-            throw new PermissionNotFoundException;
+        if (!$permission) {
+                    throw new PermissionNotFoundException;
+        }
 
         return $permission;
     }