z1haze / laravel-acl

src/migrations/2017_01_22_000001_create_levels_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('levels', function (Blueprint $table) {
+        Schema::create('levels', function(Blueprint $table) {
             $table->increments('id');
             $table->string('name')->unique();
             $table->string('label')->nullable();

src/migrations/2017_01_22_000003_add_fields_to_users_table.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
      */
     public function up()
     {
-        Schema::table(config('laravel-acl.authTable'), function (Blueprint $table) {
+        Schema::table(config('laravel-acl.authTable'), function(Blueprint $table) {
             $table->integer('level_id')->unsigned()->nullable();
 
             $table->index('level_id');
@@ -27,7 +27,7 @@ 
      */
     public function down()
     {
-        Schema::table(config('laravel-acl.authTable'), function (Blueprint $table) {
+        Schema::table(config('laravel-acl.authTable'), function(Blueprint $table) {
             $table->dropForeign(['level_id']);
             $table->dropColumn('level_id');
         });

src/migrations/2017_01_22_000005_create_permission_user_table.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
      */
     public function up()
     {
-        Schema::create('permission_' . str_singular(config('laravel-acl.authTable')), function (Blueprint $table) {
+        Schema::create('permission_' . str_singular(config('laravel-acl.authTable')), function(Blueprint $table) {
             $table->increments('id');
             $table->integer('permission_id')->unsigned();
             $table->integer(str_singular(config('laravel-acl.authTable')) . '_id')->unsigned();
@@ -36,7 +36,7 @@ 
      */
     public function down()
     {
-        Schema::table('permission_' . str_singular(config('laravel-acl.authTable')), function (Blueprint $table) {
+        Schema::table('permission_' . str_singular(config('laravel-acl.authTable')), function(Blueprint $table) {
             $table->dropForeign(['permission_id']);
             $table->dropForeign([str_singular(config('laravel-acl.authTable')) . '_id']);
         });

src/migrations/2017_01_22_000004_create_permissions_table.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::create('permissions', function (Blueprint $table) {
+        Schema::create('permissions', function(Blueprint $table) {
             $table->increments('id');
             $table->string('name')->unique();
             $table->string('label')->nullable();

src/migrations/2017_01_22_000002_create_testing_users_table.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     public function up()
     {
         if (!Schema::hasTable('users')) {
-            Schema::create('users', function (Blueprint $table) {
+            Schema::create('users', function(Blueprint $table) {
                 $table->increments('id');
                 $table->string('name');
                 $table->string('email')->unique();

src/z1haze/Acl/Models/Permission.php

Spacing +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
     {
         parent::boot();
 
-        static::deleting(function ($permission) {
+        static::deleting(function($permission) {
             $permission->users()->detach();
             $permission->level()->dissociate()->save();
         });

Braces +6 added lines, -4 removed lines

@@ -99,11 +99,13 @@
      */
     protected function getUser($user)
     {
-        if (is_int($user))
-            $user = config('laravel-acl.user')::find($user);
+        if (is_int($user)) {
+                    $user = config('laravel-acl.user')::find($user);
+        }
 
-        if (!$user)
-            throw new UserNotFoundException;
+        if (!$user) {
+                    throw new UserNotFoundException;
+        }
 
         return $user;
     }

src/z1haze/Acl/Models/Level.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
     {
         parent::boot();
 
-        static::deleting(function ($level) {
+        static::deleting(function($level) {
             $level->permissions()->where('level_id', $level->id)->update(['level_id' => null]);
             foreach ($level->users as $user) {
                 $user->level()->dissociate()->save();

src/z1haze/Acl/Traits/UserOnly.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
         return \Cache::remember(
             'laravel-acl.getNegatedPermissionsForUser_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetNegatedPermissions();
             }
         );

src/z1haze/Acl/Traits/UserAndLevel.php

Spacing +17 added lines, -26 removed lines

@@ -21,8 +21,7 @@ 
         $level = $this->getLevel($level);
 
         return get_class($this) == config('laravel-acl.level') ?
-            $this->name == $level->name :
-            $this->level->name == $level->name;
+            $this->name == $level->name : $this->level->name == $level->name;
     }
 
     /**
@@ -38,8 +37,7 @@ 
         $level = $this->getLevel($level);
 
         return get_class($this) == config('laravel-acl.level') ?
-            $this->rank <= $level->rank :
-            $this->level->rank <= $level->rank;
+            $this->rank <= $level->rank : $this->level->rank <= $level->rank;
     }
 
     /**
@@ -133,7 +131,7 @@ 
         return \Cache::remember(
             'laravel-acl.getAllPermissionsFor' . $model . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetAllPermissions();
             }
         );
@@ -155,7 +153,7 @@ 
         return \Cache::remember(
             'laravel-acl.getInheritedPermissionsFor' . $model . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetInheritedPermissions();
             }
         );
@@ -177,7 +175,7 @@ 
         return \Cache::remember(
             'laravel-acl.getAvailablePermissionsFor' . $model . '_' . $this->id,
             config('laravel-acl.cacheMinutes'),
-            function () {
+            function() {
                 return $this->cacheGetAvailablePermissions();
             }
         );
@@ -200,7 +198,7 @@ 
         return \Cache::remember(
             'laravel-acl.hasPermissionTo_' . $model . '_' . $this->id . '_Permission_' . $permission,
             config('laravel-acl.cacheMinutes'),
-            function () use ($permission) {
+            function() use ($permission) {
                 return $this->cacheHasPermissionTo($permission);
             }
         );
@@ -217,8 +215,7 @@ 
     public function isLowerThan($other)
     {
         return get_class($this) == config('laravel-acl.user') ?
-            $this->level->rank > $other->level->rank :
-            $this->rank > $other->rank;
+            $this->level->rank > $other->level->rank : $this->rank > $other->rank;
     }
 
     /**
@@ -232,8 +229,7 @@ 
     public function isHigherThan($other)
     {
         return get_class($this) == config('laravel-acl.user') ?
-            $this->level->rank < $other->level->rank :
-            $this->rank < $other->rank;
+            $this->level->rank < $other->level->rank : $this->rank < $other->rank;
     }
 
     /**
@@ -247,8 +243,7 @@ 
     public function isEqualTo($other)
     {
         return get_class($this) == config('laravel-acl.user') ?
-            $this->level->rank == $other->level->rank :
-            $this->rank == $other->rank;
+            $this->level->rank == $other->level->rank : $this->rank == $other->rank;
     }
 
 
@@ -267,14 +262,13 @@ 
     protected function cacheGetAllPermissions()
     {
         $rank = get_class($this) == config('laravel-acl.level') ?
-            $this->rank :
-            $this->level->rank;
+            $this->rank : $this->level->rank;
 
         $levels = config('laravel-acl.level', Level::class)::where('rank', '>=', $rank)->with('permissions')->get();
 
         $allPerms = new Collection();
 
-        $levels->each(function ($level, $key) use (&$allPerms) {
+        $levels->each(function($level, $key) use (&$allPerms) {
             $allPerms = $allPerms->merge($level->permissions);
         });
 
@@ -290,16 +284,14 @@ 
     protected function cacheGetInheritedPermissions()
     {
         $rank = get_class($this) == config('laravel-acl.level') ?
-            $this->rank :
-            $this->level->rank;
+            $this->rank : $this->level->rank;
 
         $levels = get_class($this) == config('laravel-acl.user') ?
-            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'));
         });
 
@@ -396,8 +388,7 @@ 
                 $permissionsObjects[$permission->id] = ['negated' => true];
             } else {
                 $model == 'User' ?
-                    array_push($permissionsObjects, $permission->id) :
-                    array_push($permissionsObjects, $permission);
+                    array_push($permissionsObjects, $permission->id) : array_push($permissionsObjects, $permission);
             }
         }
 
@@ -433,7 +424,7 @@ 
             $this->permissions()->detach($permissionObjects);
 
         if ($model == '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 +445,7 @@ 
         if ($model == '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

@@ -105,11 +105,13 @@ 
      */
     public function clearPermissions()
     {
-        if (get_class($this) == config('laravel-acl.user'))
-            $this->permissions()->detach();
+        if (get_class($this) == config('laravel-acl.user')) {
+                    $this->permissions()->detach();
+        }
 
-        if (get_class($this) == config('laravel-acl.level'))
-            config('laravel-acl.permission', Permission::class)::whereLevelId($this->id)->update(['level_id' => null]);
+        if (get_class($this) == config('laravel-acl.level')) {
+                    config('laravel-acl.permission', Permission::class)::whereLevelId($this->id)->update(['level_id' => null]);
+        }
     }
 
 
@@ -413,11 +415,13 @@ 
      */
     protected function addOrNegate($model, $permissionObjects, $action)
     {
-        if ($model == 'User')
-            $this->permissions()->attach($permissionObjects);
+        if ($model == 'User') {
+                    $this->permissions()->attach($permissionObjects);
+        }
 
-        if ($model == 'Level')
-            $this->permissions()->saveMany($permissionObjects);
+        if ($model == 'Level') {
+                    $this->permissions()->saveMany($permissionObjects);
+        }
     }
 
     /**
@@ -429,8 +433,9 @@ 
      */
     protected function remove($model, $permissionObjects)
     {
-        if ($model == 'User')
-            $this->permissions()->detach($permissionObjects);
+        if ($model == 'User') {
+                    $this->permissions()->detach($permissionObjects);
+        }
 
         if ($model == 'Level') {
             config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) {
@@ -448,8 +453,9 @@ 
      */
     protected function sync($model, $permissionObjects)
     {
-        if ($model == 'User')
-            $this->permissions()->sync($permissionObjects);
+        if ($model == 'User') {
+                    $this->permissions()->sync($permissionObjects);
+        }
 
         if ($model == '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;
     }