z1haze / laravel-acl

src/z1haze/Acl/Models/Level.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      * LEVEL
      * A Level has many users
      *
-     * @return mixed
+     * @return \Illuminate\Database\Eloquent\Relations\HasMany
      */
     public function users()
     {

src/z1haze/Acl/Models/Permission.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      * PERMISSION
      * A permission belongs to many users
      *
-     * @return mixed
+     * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
      */
     public function users()
     {

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/Traits/UserAndLevel.php

Doc Comments +7 added lines, -7 removed lines

@@ -262,7 +262,7 @@ 
      * associated to a user by direct or
      * inheritance
      *
-     * @return mixed
+     * @return Collection
      */
     protected function cacheGetAllPermissions()
     {
@@ -285,7 +285,7 @@ 
      * USER & LEVEL
      * Return a collection of permissions that
      * are inherited from a higher level
-     * @return mixed
+     * @return Collection
      */
     protected function cacheGetInheritedPermissions()
     {
@@ -347,7 +347,7 @@ 
      * Modify a given set of permissions, given the action to be taken
      *
      * @param $permissions
-     * @param $action
+     * @param string $action
      * @return mixed
      */
     protected function modifyPermissions($permissions, $action)
@@ -380,7 +380,7 @@ 
      *
      * @param $permissions
      * @param $action
-     * @param $model
+     * @param string $model
      * @return mixed
      * @throws PermissionNotFoundException
      */
@@ -408,7 +408,7 @@ 
      * USER & LEVEL
      * Helper function to handle adding or negating permissions
      *
-     * @param $model
+     * @param string $model
      * @param $permissionObjects
      */
     protected function addOrNegate($model, $permissionObjects, $action)
@@ -424,7 +424,7 @@ 
      * USER & LEVEL
      * Helper function to handle removing permissions
      *
-     * @param $model
+     * @param string $model
      * @param $permissionObjects
      */
     protected function remove($model, $permissionObjects)
@@ -443,7 +443,7 @@ 
      * USER & LEVEL
      * Help function to handle syncing permissions
      *
-     * @param $model
+     * @param string $model
      * @param $permissionObjects
      */
     protected function sync($model, $permissionObjects)

Braces +27 added lines, -18 removed lines

@@ -105,11 +105,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]);
+        }
     }
 
 
@@ -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;
     }

src/z1haze/Acl/Traits/UserAndPermission.php

Braces +9 added lines, -6 removed lines

@@ -59,14 +59,17 @@
      */
     protected function getLevel($level)
     {
-        if (is_string($level))
-            $level = config('laravel-acl.level', Level::class)::whereName($level)->first();
+        if (is_string($level)) {
+                    $level = config('laravel-acl.level', Level::class)::whereName($level)->first();
+        }
 
-        if (is_int($level))
-            $level = config('laravel-acl.level', Level::class)::find($level);
+        if (is_int($level)) {
+                    $level = config('laravel-acl.level', Level::class)::find($level);
+        }
 
-        if (!$level)
-            throw new LevelNotFoundException();
+        if (!$level) {
+                    throw new LevelNotFoundException();
+        }
 
         return $level;
     }