@@ -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 | } |