EmilMoe / Guardian

src/Http/Traits/WithPermissions.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use EmilMoe\Guardian\Http\Models\Role;
 use EmilMoe\Guardian\Support\Guardian;
 use EmilMoe\Guardian\Http\Models\Permission;
-
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Auth;

Braces +33 added lines, -22 removed lines

@@ -32,8 +32,9 @@ 
     {
         $p = collect([]);
 
-        foreach ($this->roles()->get() as $role)
-            $p = $p->merge($role->permissions()->get());
+        foreach ($this->roles()->get() as $role) {
+                    $p = $p->merge($role->permissions()->get());
+        }
 
         return $p->unique();
     }
@@ -49,14 +50,17 @@ 
      */
     public function hasAccess($permission, $id = null)
     {
-        if ($this->hasGlobalAccess($permission))
-            return true;
+        if ($this->hasGlobalAccess($permission)) {
+                    return true;
+        }
 
-        if ($this->hasLocalAccess($permission, $id))
-            return true;
+        if ($this->hasLocalAccess($permission, $id)) {
+                    return true;
+        }
 
-        if ($this->hasInheritAccess($permission, $id))
-            return true;
+        if ($this->hasInheritAccess($permission, $id)) {
+                    return true;
+        }
 
         return false;
     }
@@ -72,13 +76,15 @@ 
      */
     public function hasAccessAny($permission)
     {
-        if ($this->hasAccess($permission))
-            return true;
+        if ($this->hasAccess($permission)) {
+                    return true;
+        }
 
         $permission = Permission::where('name', $permission);
 
-        if ($permission->count() == 0 || $permission->first()->table == null)
-            return false;
+        if ($permission->count() == 0 || $permission->first()->table == null) {
+                    return false;
+        }
 
         return DB::table(Permission::where('name', $permission)->first()->table)
             ->where($permission->first()->user_id_column, Auth::id())
@@ -108,20 +114,23 @@ 
      */
     private function hasLocalAccess($permission, $id)
     {
-        if (! $id)
-            return false;
+        if (! $id) {
+                    return false;
+        }
 
         $permission = Permission::where('name', $permission);
 
-        if ($permission->count() == 0 || $permission->first()->table == null)
-            return false;
+        if ($permission->count() == 0 || $permission->first()->table == null) {
+                    return false;
+        }
 
         if (DB::table($permission->first()->table)
                 ->where($permission->first()->user_id_column, Auth::id())
                 ->where($permission->first()->foreign_id_column, $id)
                 ->where('is_privileged', true)
-                ->count() > 0)
-            return true;
+                ->count() > 0) {
+                    return true;
+        }
 
         return false;
     }
@@ -138,11 +147,13 @@ 
      */
     private function hasInheritAccess($permission, $id)
     {
-        if (! $id)
-            return false;
+        if (! $id) {
+                    return false;
+        }
 
-        if (method_exists(Auth::user(), $permission .'Privilege'))
-            return Auth::user()->{$permission .'Privilege'}($id);
+        if (method_exists(Auth::user(), $permission .'Privilege')) {
+                    return Auth::user()->{$permission .'Privilege'}($id);
+        }
 
         return false;
     }

src/database/migrations/2016_05_03_141741_create_guardian_tables.php

Braces +6 added lines, -4 removed lines

@@ -28,16 +28,18 @@
             $table->string('name');
             $table->boolean('locked')->default(false);
 
-            if (Guardian::hasClients())
-                $table->integer(Guardian::getClientColumn())->unsigned();
+            if (Guardian::hasClients()) {
+                            $table->integer(Guardian::getClientColumn())->unsigned();
+            }
 
             $table->timestamps();
 
-            if (Guardian::hasClients())
-                $table->foreign(Guardian::getClientColumn())
+            if (Guardian::hasClients()) {
+                            $table->foreign(Guardian::getClientColumn())
                     ->references(Guardian::getClientKey())
                     ->on(Guardian::getClientTable())
                     ->onDelete('cascade');
+            }
         });
 
         Schema::create(config('guardian.table.role') .'_'. config('guardian.table.permission'), function (Blueprint $table) {

src/Support/Guardian.php

Braces +3 added lines, -2 removed lines

@@ -65,8 +65,9 @@
     {
         return Auth::user()->{config('guardian.client.relation')};
 
-        foreach (config('guardian.client') as $step)
-            $user = $user->{$step};
+        foreach (config('guardian.client') as $step) {
+                    $user = $user->{$step};
+        }
 
         return $user;
     }

src/Http/Middleware/GuardianMiddleware.php

Braces +21 added lines, -14 removed lines

@@ -22,8 +22,9 @@ 
     {
         $this->toCollection($permission);
 
-        if (! $this->hasAccess($request, $permission))
-            return Guardian::restricted();
+        if (! $this->hasAccess($request, $permission)) {
+                    return Guardian::restricted();
+        }
 
         return $next($request);
     }
@@ -50,11 +51,13 @@ 
      */
     private function hasAccess($request, $permission)
     {
-        if (! Auth::check())
-            return false;
+        if (! Auth::check()) {
+                    return false;
+        }
 
-        if ($this->hasGlobalAccess($permission) || $this->hasLocalAccess($request, $permission))
-            return true;
+        if ($this->hasGlobalAccess($permission) || $this->hasLocalAccess($request, $permission)) {
+                    return true;
+        }
 
         return false;
     }
@@ -67,12 +70,14 @@ 
      */
     private function hasGlobalAccess($permission)
     {
-        if (is_string($permission))
-            return Auth::user()->hasAccess($permission);
+        if (is_string($permission)) {
+                    return Auth::user()->hasAccess($permission);
+        }
 
-        foreach ($permission as $perm)
-            if ($this->hasGlobalAccess($perm->getName()))
+        foreach ($permission as $perm) {
+                    if ($this->hasGlobalAccess($perm->getName()))
                 return true;
+        }
 
         return false;
     }
@@ -87,12 +92,14 @@ 
      */
     private function hasLocalAccess($request, $permission, $id = null)
     {
-        if (is_string($permission))
-            return Auth::user()->hasAccess($permission, $id);
+        if (is_string($permission)) {
+                    return Auth::user()->hasAccess($permission, $id);
+        }
 
-        foreach ($permission as $perm)
-            if ($this->hasLocalAccess($request, $perm->getName(), $request->{$perm->getColumn()}))
+        foreach ($permission as $perm) {
+                    if ($this->hasLocalAccess($request, $perm->getName(), $request->{$perm->getColumn()}))
                 return true;
+        }
 
         return false;
     }

src/Http/Models/Role.php

Braces +42 added lines, -28 removed lines

@@ -37,8 +37,9 @@ 
      */
     public static function create(array $attributes = [])
     {
-        if (Guardian::hasClients())
-            $attributes[Guardian::getClientColumn()] = Guardian::getClientId();
+        if (Guardian::hasClients()) {
+                    $attributes[Guardian::getClientColumn()] = Guardian::getClientId();
+        }
 
         return parent::create($attributes);
     }
@@ -85,11 +86,13 @@ 
      */
     public function update(array $attributes = [], array $options = [])
     {
-        if (Guardian::hasClients())
-            $attributes[Guardian::getClientColumn()] = Guardian::getClientId();
+        if (Guardian::hasClients()) {
+                    $attributes[Guardian::getClientColumn()] = Guardian::getClientId();
+        }
 
-        if ($this->locked)
-            return false;
+        if ($this->locked) {
+                    return false;
+        }
 
         return parent::update($attributes, $options);
     }
@@ -104,15 +107,18 @@ 
      */
     public function addUser($userid)
     {
-        if ($this->locked == true)
-            return;
+        if ($this->locked == true) {
+                    return;
+        }
 
-        if (Guardian::hasClients())
-            if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
+        if (Guardian::hasClients()) {
+                    if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
                 return;
+        }
 
-        if (! $this->users()->get()->contains($userid))
-            $this->users()->attach($userid);
+        if (! $this->users()->get()->contains($userid)) {
+                    $this->users()->attach($userid);
+        }
     }
 
     /**
@@ -125,12 +131,14 @@ 
      */
     public function removeUser($userid)
     {
-        if ($this->locked == true)
-            return;
+        if ($this->locked == true) {
+                    return;
+        }
 
-        if (Guardian::hasClients())
-            if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
+        if (Guardian::hasClients()) {
+                    if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
                 return;
+        }
 
         $this->users()->detach($userid);
     }
@@ -145,17 +153,20 @@ 
      */
     public function addPermission($permission)
     {
-        if ($this->locked == true)
-            return;
+        if ($this->locked == true) {
+                    return;
+        }
 
-        if (Guardian::hasClients())
-            if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
+        if (Guardian::hasClients()) {
+                    if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
                 return;
+        }
 
         $id = Permission::where('name', $permission)->first()->id;
 
-        if (! $this->permissions()->get()->contains($id))
-            $this->permissions()->attach($id);
+        if (! $this->permissions()->get()->contains($id)) {
+                    $this->permissions()->attach($id);
+        }
     }
 
     /**
@@ -168,12 +179,14 @@ 
      */
     public function removePermission($permission)
     {
-        if ($this->locked == true)
-            return;
+        if ($this->locked == true) {
+                    return;
+        }
 
-        if (Guardian::hasClients())
-            if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
+        if (Guardian::hasClients()) {
+                    if (Guardian::getClientId() != $this->{Guardian::getClientColumn()})
                 return;
+        }
 
         $id = Permission::where('name', $permission)->first()->id;
 
@@ -188,8 +201,9 @@ 
      */
     public function scopeClient($query)
     {
-        if (! Guardian::hasClients())
-            return $query;
+        if (! Guardian::hasClients()) {
+                    return $query;
+        }
 
         return $query->where('client_id', Guardian::getClientId());
     }

src/GuardianServiceProvider.php

Braces +3 added lines, -2 removed lines

@@ -35,13 +35,14 @@
         $router = $this->app['router'];
         $router->middleware('guard', GuardianMiddleware::class);
 
-        if (config('guardian.api.enabled'))
-            $router->group([
+        if (config('guardian.api.enabled')) {
+                    $router->group([
                 'namespace'  => 'EmilMoe\Guardian\Http\Controllers',
                 'prefix'     => config('guardian.api.url'),
                 'middleware' => 'auth'
             ], function () {
                 require __DIR__ .'/Http/routes.php';
+        }
             });
     }
 }
\ No newline at end of file