@@ -21,8 +21,7 @@ discard block |
||
| 21 | 21 | $level = $this->getLevel($level); |
| 22 | 22 | |
| 23 | 23 | return is_a($this, config('laravel-acl.level'), true) ? |
| 24 | - $this->name == $level->name : |
|
| 25 | - $this->level->name == $level->name; |
|
| 24 | + $this->name == $level->name : $this->level->name == $level->name; |
|
| 26 | 25 | } |
| 27 | 26 | |
| 28 | 27 | /** |
@@ -38,8 +37,7 @@ discard block |
||
| 38 | 37 | $level = $this->getLevel($level); |
| 39 | 38 | |
| 40 | 39 | return is_a($this, config('laravel-acl.level'), true) ? |
| 41 | - $this->rank <= $level->rank : |
|
| 42 | - $this->level->rank <= $level->rank; |
|
| 40 | + $this->rank <= $level->rank : $this->level->rank <= $level->rank; |
|
| 43 | 41 | } |
| 44 | 42 | |
| 45 | 43 | /** |
@@ -133,7 +131,7 @@ discard block |
||
| 133 | 131 | return \Cache::remember( |
| 134 | 132 | 'laravel-acl.getAllPermissionsFor' . $model . '_' . $this->id, |
| 135 | 133 | config('laravel-acl.cacheMinutes'), |
| 136 | - function () { |
|
| 134 | + function() { |
|
| 137 | 135 | return $this->cacheGetAllPermissions(); |
| 138 | 136 | } |
| 139 | 137 | ); |
@@ -155,7 +153,7 @@ discard block |
||
| 155 | 153 | return \Cache::remember( |
| 156 | 154 | 'laravel-acl.getInheritedPermissionsFor' . $model . '_' . $this->id, |
| 157 | 155 | config('laravel-acl.cacheMinutes'), |
| 158 | - function () { |
|
| 156 | + function() { |
|
| 159 | 157 | return $this->cacheGetInheritedPermissions(); |
| 160 | 158 | } |
| 161 | 159 | ); |
@@ -177,7 +175,7 @@ discard block |
||
| 177 | 175 | return \Cache::remember( |
| 178 | 176 | 'laravel-acl.getAvailablePermissionsFor' . $model . '_' . $this->id, |
| 179 | 177 | config('laravel-acl.cacheMinutes'), |
| 180 | - function () { |
|
| 178 | + function() { |
|
| 181 | 179 | return $this->cacheGetAvailablePermissions(); |
| 182 | 180 | } |
| 183 | 181 | ); |
@@ -200,7 +198,7 @@ discard block |
||
| 200 | 198 | return \Cache::remember( |
| 201 | 199 | 'laravel-acl.hasPermissionTo_' . $model . '_' . $this->id . '_Permission_' . $permission, |
| 202 | 200 | config('laravel-acl.cacheMinutes'), |
| 203 | - function () use ($permission) { |
|
| 201 | + function() use ($permission) { |
|
| 204 | 202 | return $this->cacheHasPermissionTo($permission); |
| 205 | 203 | } |
| 206 | 204 | ); |
@@ -217,8 +215,7 @@ discard block |
||
| 217 | 215 | public function isLowerThan($other) |
| 218 | 216 | { |
| 219 | 217 | return is_a($this, config('laravel-acl.user'), true) ? |
| 220 | - $this->level->rank > $other->level->rank : |
|
| 221 | - $this->rank > $other->rank; |
|
| 218 | + $this->level->rank > $other->level->rank : $this->rank > $other->rank; |
|
| 222 | 219 | } |
| 223 | 220 | |
| 224 | 221 | /** |
@@ -232,8 +229,7 @@ discard block |
||
| 232 | 229 | public function isHigherThan($other) |
| 233 | 230 | { |
| 234 | 231 | return is_a($this, config('laravel-acl.user'), true) ? |
| 235 | - $this->level->rank < $other->level->rank : |
|
| 236 | - $this->rank < $other->rank; |
|
| 232 | + $this->level->rank < $other->level->rank : $this->rank < $other->rank; |
|
| 237 | 233 | } |
| 238 | 234 | |
| 239 | 235 | /** |
@@ -247,8 +243,7 @@ discard block |
||
| 247 | 243 | public function isEqualTo($other) |
| 248 | 244 | { |
| 249 | 245 | return is_a($this, config('laravel-acl.user'), true) ? |
| 250 | - $this->level->rank == $other->level->rank : |
|
| 251 | - $this->rank == $other->rank; |
|
| 246 | + $this->level->rank == $other->level->rank : $this->rank == $other->rank; |
|
| 252 | 247 | } |
| 253 | 248 | |
| 254 | 249 | |
@@ -267,14 +262,13 @@ discard block |
||
| 267 | 262 | protected function cacheGetAllPermissions() |
| 268 | 263 | { |
| 269 | 264 | $rank = is_a($this, config('laravel-acl.level'), true) ? |
| 270 | - $this->rank : |
|
| 271 | - $this->level->rank; |
|
| 265 | + $this->rank : $this->level->rank; |
|
| 272 | 266 | |
| 273 | 267 | $levels = config('laravel-acl.level', Level::class)::where('rank', '>=', $rank)->with('permissions')->get(); |
| 274 | 268 | |
| 275 | 269 | $allPerms = new Collection(); |
| 276 | 270 | |
| 277 | - $levels->each(function ($level, $key) use (&$allPerms) { |
|
| 271 | + $levels->each(function($level, $key) use (&$allPerms) { |
|
| 278 | 272 | $allPerms = $allPerms->merge($level->permissions); |
| 279 | 273 | }); |
| 280 | 274 | |
@@ -290,16 +284,14 @@ discard block |
||
| 290 | 284 | protected function cacheGetInheritedPermissions() |
| 291 | 285 | { |
| 292 | 286 | $rank = is_a($this, config('laravel-acl.level'), true) ? |
| 293 | - $this->rank : |
|
| 294 | - $this->level->rank; |
|
| 287 | + $this->rank : $this->level->rank; |
|
| 295 | 288 | |
| 296 | 289 | $levels = is_a($this, config('laravel-acl.user'), true) ? |
| 297 | - config('laravel-acl.level', Level::class)::where('rank', '>=', $rank)->with('permissions')->get() : |
|
| 298 | - config('laravel-acl.level', Level::class)::where('rank', '>', $rank)->with('permissions')->get(); |
|
| 290 | + config('laravel-acl.level', Level::class)::where('rank', '>=', $rank)->with('permissions')->get() : config('laravel-acl.level', Level::class)::where('rank', '>', $rank)->with('permissions')->get(); |
|
| 299 | 291 | |
| 300 | 292 | $inheritedPerms = new Collection(); |
| 301 | 293 | |
| 302 | - $levels->each(function ($level, $key) use (&$inheritedPerms) { |
|
| 294 | + $levels->each(function($level, $key) use (&$inheritedPerms) { |
|
| 303 | 295 | $inheritedPerms = $inheritedPerms->merge($level->permissions->load('level')); |
| 304 | 296 | }); |
| 305 | 297 | |
@@ -396,8 +388,7 @@ discard block |
||
| 396 | 388 | $permissionsObjects[$permission->id] = ['negated' => true]; |
| 397 | 389 | } else { |
| 398 | 390 | $model == 'User' ? |
| 399 | - array_push($permissionsObjects, $permission->id) : |
|
| 400 | - array_push($permissionsObjects, $permission); |
|
| 391 | + array_push($permissionsObjects, $permission->id) : array_push($permissionsObjects, $permission); |
|
| 401 | 392 | } |
| 402 | 393 | } |
| 403 | 394 | |
@@ -433,7 +424,7 @@ discard block |
||
| 433 | 424 | $this->permissions()->detach($permissionObjects); |
| 434 | 425 | |
| 435 | 426 | if ($model == 'Level') { |
| 436 | - config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) { |
|
| 427 | + config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function($permission) { |
|
| 437 | 428 | return $permission->id; |
| 438 | 429 | }, $permissionObjects))->update(['level_id' => null]); |
| 439 | 430 | } |
@@ -454,7 +445,7 @@ discard block |
||
| 454 | 445 | if ($model == 'Level') { |
| 455 | 446 | $this->clearPermissions(); |
| 456 | 447 | |
| 457 | - config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) { |
|
| 448 | + config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function($permission) { |
|
| 458 | 449 | return $permission->id; |
| 459 | 450 | }, $permissionObjects)) |
| 460 | 451 | ->update(['level_id' => $this->id]); |
@@ -105,11 +105,13 @@ discard block |
||
| 105 | 105 | */ |
| 106 | 106 | public function clearPermissions() |
| 107 | 107 | { |
| 108 | - if (is_a($this, config('laravel-acl.user'), true)) |
|
| 109 | - $this->permissions()->detach(); |
|
| 108 | + if (is_a($this, config('laravel-acl.user'), true)) { |
|
| 109 | + $this->permissions()->detach(); |
|
| 110 | + } |
|
| 110 | 111 | |
| 111 | - if (is_a($this, config('laravel-acl.level'), true)) |
|
| 112 | - config('laravel-acl.permission', Permission::class)::whereLevelId($this->id)->update(['level_id' => null]); |
|
| 112 | + if (is_a($this, config('laravel-acl.level'), true)) { |
|
| 113 | + config('laravel-acl.permission', Permission::class)::whereLevelId($this->id)->update(['level_id' => null]); |
|
| 114 | + } |
|
| 113 | 115 | } |
| 114 | 116 | |
| 115 | 117 | |
@@ -413,11 +415,13 @@ discard block |
||
| 413 | 415 | */ |
| 414 | 416 | protected function addOrNegate($model, $permissionObjects, $action) |
| 415 | 417 | { |
| 416 | - if ($model == 'User') |
|
| 417 | - $this->permissions()->attach($permissionObjects); |
|
| 418 | + if ($model == 'User') { |
|
| 419 | + $this->permissions()->attach($permissionObjects); |
|
| 420 | + } |
|
| 418 | 421 | |
| 419 | - if ($model == 'Level') |
|
| 420 | - $this->permissions()->saveMany($permissionObjects); |
|
| 422 | + if ($model == 'Level') { |
|
| 423 | + $this->permissions()->saveMany($permissionObjects); |
|
| 424 | + } |
|
| 421 | 425 | } |
| 422 | 426 | |
| 423 | 427 | /** |
@@ -429,8 +433,9 @@ discard block |
||
| 429 | 433 | */ |
| 430 | 434 | protected function remove($model, $permissionObjects) |
| 431 | 435 | { |
| 432 | - if ($model == 'User') |
|
| 433 | - $this->permissions()->detach($permissionObjects); |
|
| 436 | + if ($model == 'User') { |
|
| 437 | + $this->permissions()->detach($permissionObjects); |
|
| 438 | + } |
|
| 434 | 439 | |
| 435 | 440 | if ($model == 'Level') { |
| 436 | 441 | config('laravel-acl.permission', Permission::class)::whereIn('id', array_map(function ($permission) { |
@@ -448,8 +453,9 @@ discard block |
||
| 448 | 453 | */ |
| 449 | 454 | protected function sync($model, $permissionObjects) |
| 450 | 455 | { |
| 451 | - if ($model == 'User') |
|
| 452 | - $this->permissions()->sync($permissionObjects); |
|
| 456 | + if ($model == 'User') { |
|
| 457 | + $this->permissions()->sync($permissionObjects); |
|
| 458 | + } |
|
| 453 | 459 | |
| 454 | 460 | if ($model == 'Level') { |
| 455 | 461 | $this->clearPermissions(); |
@@ -472,14 +478,17 @@ discard block |
||
| 472 | 478 | */ |
| 473 | 479 | protected function getPermission($permission) |
| 474 | 480 | { |
| 475 | - if (is_string($permission)) |
|
| 476 | - $permission = config('laravel-acl.permission', Permission::class)::whereName($permission)->first(); |
|
| 481 | + if (is_string($permission)) { |
|
| 482 | + $permission = config('laravel-acl.permission', Permission::class)::whereName($permission)->first(); |
|
| 483 | + } |
|
| 477 | 484 | |
| 478 | - if (is_int($permission)) |
|
| 479 | - $permission = config('laravel-acl.permission', Permission::class)::find($permission); |
|
| 485 | + if (is_int($permission)) { |
|
| 486 | + $permission = config('laravel-acl.permission', Permission::class)::find($permission); |
|
| 487 | + } |
|
| 480 | 488 | |
| 481 | - if (!$permission) |
|
| 482 | - throw new PermissionNotFoundException; |
|
| 489 | + if (!$permission) { |
|
| 490 | + throw new PermissionNotFoundException; |
|
| 491 | + } |
|
| 483 | 492 | |
| 484 | 493 | return $permission; |
| 485 | 494 | } |