HasPermissions - Code Metrics - Inspection of "add type checking in hasPermissionTo and hasDirect..." - spatie/laravel-permission - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#820)

unknown

created 2018-08-08 00:42 UTC

HasPermissions B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	338
Duplicated Lines	21.6 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

Metric	Value
dl	73
loc	338
rs	8.8
c	0
b	0
f	0
wmc	45
lcom	1
cbo	6

19 Methods

Rating	Name	Duplication	Size	Complexity
A	bootHasPermissions()	0	10	3
A	permissions()	0	10	1
A	scopePermission()	0	27	4
A	convertToPermissionModels()	0	16	3
A	hasPermissionTo()	12	22	5
A	hasAnyPermission()	14	14	4
A	hasAllPermissions()	14	14	4
A	hasPermissionViaRole()	0	4	1
B	hasDirectPermission()	12	22	6
A	getPermissionsViaRoles()	0	7	1
A	getAllPermissions()	0	7	1
A	givePermissionTo()	21	21	1
A	syncPermissions()	0	6	1
A	revokePermissionTo()	0	8	1
A	getStoredPermission()	0	19	4
A	ensureModelSharesGuard()	0	6	2
A	getGuardNames()	0	4	1
A	getDefaultGuardName()	0	4	1
A	forgetCachedPermissions()	0	4	1

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like HasPermissions often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use HasPermissions, and based on these observations, apply Extract Interface, too.

<?php

namespace Spatie\Permission\Traits;

use Spatie\Permission\Guard;
use Illuminate\Support\Collection;
use Illuminate\Database\Eloquent\Builder;
use Spatie\Permission\PermissionRegistrar;
use Spatie\Permission\Contracts\Permission;
use Spatie\Permission\Exceptions\GuardDoesNotMatch;
use Illuminate\Database\Eloquent\Relations\MorphToMany;
use Spatie\Permission\Exceptions\PermissionDoesNotExist;

trait HasPermissions
{
    public static function bootHasPermissions()
    {
        static::deleting(function ($model) {
            if (method_exists($model, 'isForceDeleting') && ! $model->isForceDeleting()) {
                return;
            }

            $model->permissions()->detach();
        });
    }

    /**
     * A model may have multiple direct permissions.
     */
    public function permissions(): MorphToMany
    {
        return $this->morphToMany(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            config('permission.models.permission'),
            'model',
            config('permission.table_names.model_has_permissions'),
            'model_id',
            'permission_id'
        );
    }

    /**
     * Scope the model query to certain permissions only.
     *
     * @param \Illuminate\Database\Eloquent\Builder $query
     * @param string|array|\Spatie\Permission\Contracts\Permission|\Illuminate\Support\Collection $permissions
     *
     * @return \Illuminate\Database\Eloquent\Builder
     */
    public function scopePermission(Builder $query, $permissions): Builder
    {
        $permissions = $this->convertToPermissionModels($permissions);

        $rolesWithPermissions = array_unique(array_reduce($permissions, function ($result, $permission) {
            return array_merge($result, $permission->roles->all());
        }, []));

        return $query->where(function ($query) use ($permissions, $rolesWithPermissions) {
            $query->whereHas('permissions', function ($query) use ($permissions) {
                $query->where(function ($query) use ($permissions) {
                    foreach ($permissions as $permission) {
                        $query->orWhere(config('permission.table_names.permissions').'.id', $permission->id);
                    }
                });
            });
            if (count($rolesWithPermissions) > 0) {
                $query->orWhereHas('roles', function ($query) use ($rolesWithPermissions) {
                    $query->where(function ($query) use ($rolesWithPermissions) {
                        foreach ($rolesWithPermissions as $role) {
                            $query->orWhere(config('permission.table_names.roles').'.id', $role->id);
                        }
                    });
                });
            }
        });
    }

    /**
     * @param string|array|\Spatie\Permission\Contracts\Permission|\Illuminate\Support\Collection $permissions
     *
     * @return array
     */
    protected function convertToPermissionModels($permissions): array
    {
        if ($permissions instanceof Collection) {
            $permissions = $permissions->all();
        }

        $permissions = array_wrap($permissions);

        return array_map(function ($permission) {
            if ($permission instanceof Permission) {
                return $permission;
            }

            return app(Permission::class)->findByName($permission, $this->getDefaultGuardName());
        }, $permissions);
    }

    /**
     * Determine if the model may perform the given permission.
     *
     * @param string|int|\Spatie\Permission\Contracts\Permission $permission
     * @param string|null $guardName
     *
     * @return bool
     */
    public function hasPermissionTo($permission, $guardName = null): bool
    {
        if (is_string($permission)) {

            $permission = app(Permission::class)->findByName(
                $permission,
                $guardName ?? $this->getDefaultGuardName()
            );
        }

        if (is_int($permission)) {

            $permission = app(Permission::class)->findById(
                $permission,
                $guardName ?? $this->getDefaultGuardName()
            );
        }

        if (! $permission instanceof Permission) {
            throw new PermissionDoesNotExist;
        }

        return $this->hasDirectPermission($permission) || $this->hasPermissionViaRole($permission);
    }

    /**
     * Determine if the model has any of the given permissions.
     *
     * @param array ...$permissions
     *
     * @return bool
     */
    public function hasAnyPermission(...$permissions): bool

    {
        if (is_array($permissions[0])) {
            $permissions = $permissions[0];
        }

        foreach ($permissions as $permission) {
            if ($this->hasPermissionTo($permission)) {
                return true;
            }
        }

        return false;
    }

    /**
     * Determine if the model has all of the given permissions.
     *
     * @param array ...$permissions
     *
     * @return bool
     */
    public function hasAllPermissions(...$permissions): bool

    {
        if (is_array($permissions[0])) {
            $permissions = $permissions[0];
        }

        foreach ($permissions as $permission) {
            if (! $this->hasPermissionTo($permission)) {
                return false;
            }
        }

        return true;
    }

    /**
     * Determine if the model has, via roles, the given permission.
     *
     * @param \Spatie\Permission\Contracts\Permission $permission
     *
     * @return bool
     */
    protected function hasPermissionViaRole(Permission $permission): bool
    {
        return $this->hasRole($permission->roles);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
    }

    /**
     * Determine if the model has the given permission.
     *
     * @param string|int|\Spatie\Permission\Contracts\Permission $permission
     *
     * @return bool
     */
    public function hasDirectPermission($permission): bool
    {
        if (is_string($permission)) {

            $permission = app(Permission::class)->findByName($permission, $this->getDefaultGuardName());
            if (! $permission) {
                return false;
            }
        }

        if (is_int($permission)) {

            $permission = app(Permission::class)->findById($permission, $this->getDefaultGuardName());
            if (! $permission) {
                return false;
            }
        }

        if (! $permission instanceof Permission) {
            return false;
        }

        return $this->permissions->contains('id', $permission->id);
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

    /**
     * Return all the permissions the model has via roles.
     */
    public function getPermissionsViaRoles(): Collection
    {
        return $this->load('roles', 'roles.permissions')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ->roles->flatMap(function ($role) {
                return $role->permissions;
            })->sort()->values();
    }

    /**
     * Return all the permissions the model has, both directly and via roles.
     */
    public function getAllPermissions(): Collection
    {
        return $this->permissions
            ->merge($this->getPermissionsViaRoles())
            ->sort()
            ->values();
    }

    /**
     * Grant the given permission(s) to a role.
     *
     * @param string|array|\Spatie\Permission\Contracts\Permission|\Illuminate\Support\Collection $permissions
     *
     * @return $this
     */
    public function givePermissionTo(...$permissions)

    {
        $permissions = collect($permissions)
            ->flatten()
            ->map(function ($permission) {
                return $this->getStoredPermission($permission);
            })
            ->filter(function ($permission) {
                return $permission instanceof Permission;
            })
            ->each(function ($permission) {
                $this->ensureModelSharesGuard($permission);
            })
            ->all();

        $this->permissions()->saveMany($permissions);

        $this->forgetCachedPermissions();

        return $this;
    }

    /**
     * Remove all current permissions and set the given ones.
     *
     * @param string|array|\Spatie\Permission\Contracts\Permission|\Illuminate\Support\Collection $permissions
     *
     * @return $this
     */
    public function syncPermissions(...$permissions)
    {
        $this->permissions()->detach();

        return $this->givePermissionTo($permissions);
    }

    /**
     * Revoke the given permission.
     *
     * @param \Spatie\Permission\Contracts\Permission|\Spatie\Permission\Contracts\Permission[]|string|string[] $permission
     *
     * @return $this
     */
    public function revokePermissionTo($permission)
    {
        $this->permissions()->detach($this->getStoredPermission($permission));

        $this->forgetCachedPermissions();

        return $this;
    }

    /**
     * @param string|array|\Spatie\Permission\Contracts\Permission|\Illuminate\Support\Collection $permissions
     *
     * @return \Spatie\Permission\Contracts\Permission|\Spatie\Permission\Contracts\Permission[]|\Illuminate\Support\Collection
     */
    protected function getStoredPermission($permissions)
    {
        if (is_numeric($permissions)) {
            return app(Permission::class)->findById($permissions, $this->getDefaultGuardName());
        }

        if (is_string($permissions)) {
            return app(Permission::class)->findByName($permissions, $this->getDefaultGuardName());
        }

        if (is_array($permissions)) {
            return app(Permission::class)
                ->whereIn('name', $permissions)
                ->whereIn('guard_name', $this->getGuardNames())
                ->get();
        }

        return $permissions;
    }

    /**
     * @param \Spatie\Permission\Contracts\Permission|\Spatie\Permission\Contracts\Role $roleOrPermission
     *
     * @throws \Spatie\Permission\Exceptions\GuardDoesNotMatch
     */
    protected function ensureModelSharesGuard($roleOrPermission)
    {
        if (! $this->getGuardNames()->contains($roleOrPermission->guard_name)) {
            throw GuardDoesNotMatch::create($roleOrPermission->guard_name, $this->getGuardNames());
        }
    }

    protected function getGuardNames(): Collection
    {
        return Guard::getNames($this);
    }

    protected function getDefaultGuardName(): string
    {
        return Guard::getDefaultName($this);
    }

    /**
     * Forget the cached permissions.
     */
    public function forgetCachedPermissions()
    {
        app(PermissionRegistrar::class)->forgetCachedPermissions();
    }
}


Pull Request — master (#820)

HasPermissions B

Complexity

Size/Duplication

Coupling/Cohesion

Importance

19 Methods

How to fix Duplicated Code Complexity

Duplicated Code

Complex Class

Available Fixes

Available Fixes

1		<?php
2
3		namespace Spatie\Permission\Traits;
4
5		use Spatie\Permission\Guard;
6		use Illuminate\Support\Collection;
7		use Illuminate\Database\Eloquent\Builder;
8		use Spatie\Permission\PermissionRegistrar;
9		use Spatie\Permission\Contracts\Permission;
10		use Spatie\Permission\Exceptions\GuardDoesNotMatch;
11		use Illuminate\Database\Eloquent\Relations\MorphToMany;
12		use Spatie\Permission\Exceptions\PermissionDoesNotExist;
13
14		trait HasPermissions
15		{
16		public static function bootHasPermissions()
17		{
18		static::deleting(function ($model) {
19		if (method_exists($model, 'isForceDeleting') && ! $model->isForceDeleting()) {
20		return;
21		}
22
23		$model->permissions()->detach();
24		});
25		}
26
27		/**
28		* A model may have multiple direct permissions.
29		*/
30		public function permissions(): MorphToMany
31		{
32		return $this->morphToMany(
		0 ignored issues – show Bug introduced 2017-04-20 14:18 UTC by Report Bug Copy Issue Report It seems like `morphToMany()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
33		config('permission.models.permission'),
34		'model',
35		config('permission.table_names.model_has_permissions'),
36		'model_id',
37		'permission_id'
38		);
39		}
40
41		/**
42		* Scope the model query to certain permissions only.
43		*
44		* @param \Illuminate\Database\Eloquent\Builder $query
45		* @param string\|array\|\Spatie\Permission\Contracts\Permission\|\Illuminate\Support\Collection $permissions
46		*
47		* @return \Illuminate\Database\Eloquent\Builder
48		*/
49		public function scopePermission(Builder $query, $permissions): Builder
50		{
51		$permissions = $this->convertToPermissionModels($permissions);
52
53		$rolesWithPermissions = array_unique(array_reduce($permissions, function ($result, $permission) {
54		return array_merge($result, $permission->roles->all());
55		}, []));
56
57		return $query->where(function ($query) use ($permissions, $rolesWithPermissions) {
58		$query->whereHas('permissions', function ($query) use ($permissions) {
59		$query->where(function ($query) use ($permissions) {
60		foreach ($permissions as $permission) {
61		$query->orWhere(config('permission.table_names.permissions').'.id', $permission->id);
62		}
63		});
64		});
65		if (count($rolesWithPermissions) > 0) {
66		$query->orWhereHas('roles', function ($query) use ($rolesWithPermissions) {
67		$query->where(function ($query) use ($rolesWithPermissions) {
68		foreach ($rolesWithPermissions as $role) {
69		$query->orWhere(config('permission.table_names.roles').'.id', $role->id);
70		}
71		});
72		});
73		}
74		});
75		}
76
77		/**
78		* @param string\|array\|\Spatie\Permission\Contracts\Permission\|\Illuminate\Support\Collection $permissions
79		*
80		* @return array
81		*/
82		protected function convertToPermissionModels($permissions): array
83		{
84		if ($permissions instanceof Collection) {
85		$permissions = $permissions->all();
86		}
87
88		$permissions = array_wrap($permissions);
89
90		return array_map(function ($permission) {
91		if ($permission instanceof Permission) {
92		return $permission;
93		}
94
95		return app(Permission::class)->findByName($permission, $this->getDefaultGuardName());
96		}, $permissions);
97		}
98
99		/**
100		* Determine if the model may perform the given permission.
101		*
102		* @param string\|int\|\Spatie\Permission\Contracts\Permission $permission
103		* @param string\|null $guardName
104		*
105		* @return bool
106		*/
107		public function hasPermissionTo($permission, $guardName = null): bool
108		{
109	View Code Duplication	if (is_string($permission)) {
		0 ignored issues – show Duplication introduced 2017-06-06 16:24 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
110		$permission = app(Permission::class)->findByName(
111		$permission,
112		$guardName ?? $this->getDefaultGuardName()
113		);
114		}
115
116	View Code Duplication	if (is_int($permission)) {
		0 ignored issues – show Duplication introduced 2018-07-05 13:05 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
117		$permission = app(Permission::class)->findById(
118		$permission,
119		$guardName ?? $this->getDefaultGuardName()
120		);
121		}
122
123		if (! $permission instanceof Permission) {
124		throw new PermissionDoesNotExist;
125		}
126
127		return $this->hasDirectPermission($permission) \|\| $this->hasPermissionViaRole($permission);
128		}
129
130		/**
131		* Determine if the model has any of the given permissions.
132		*
133		* @param array ...$permissions
134		*
135		* @return bool
136		*/
137	View Code Duplication	public function hasAnyPermission(...$permissions): bool
		0 ignored issues – show Duplication introduced 2018-06-14 02:51 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
138		{
139		if (is_array($permissions[0])) {
140		$permissions = $permissions[0];
141		}
142
143		foreach ($permissions as $permission) {
144		if ($this->hasPermissionTo($permission)) {
145		return true;
146		}
147		}
148
149		return false;
150		}
151
152		/**
153		* Determine if the model has all of the given permissions.
154		*
155		* @param array ...$permissions
156		*
157		* @return bool
158		*/
159	View Code Duplication	public function hasAllPermissions(...$permissions): bool
		0 ignored issues – show Duplication introduced 2018-06-14 02:51 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
160		{
161		if (is_array($permissions[0])) {
162		$permissions = $permissions[0];
163		}
164
165		foreach ($permissions as $permission) {
166		if (! $this->hasPermissionTo($permission)) {
167		return false;
168		}
169		}
170
171		return true;
172		}
173
174		/**
175		* Determine if the model has, via roles, the given permission.
176		*
177		* @param \Spatie\Permission\Contracts\Permission $permission
178		*
179		* @return bool
180		*/
181		protected function hasPermissionViaRole(Permission $permission): bool
182		{
183		return $this->hasRole($permission->roles);
		0 ignored issues – show Bug introduced 2015-12-21 23:32 UTC by Report Bug Copy Issue Report Accessing `roles` on the interface `Spatie\Permission\Contracts\Permission` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history... Bug introduced 2018-02-05 03:35 UTC by Report Bug Copy Issue Report It seems like `hasRole()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
184		}
185
186		/**
187		* Determine if the model has the given permission.
188		*
189		* @param string\|int\|\Spatie\Permission\Contracts\Permission $permission
190		*
191		* @return bool
192		*/
193		public function hasDirectPermission($permission): bool
194		{
195	View Code Duplication	if (is_string($permission)) {
		0 ignored issues – show Duplication introduced 2017-06-06 16:24 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
196		$permission = app(Permission::class)->findByName($permission, $this->getDefaultGuardName());
197		if (! $permission) {
198		return false;
199		}
200		}
201
202	View Code Duplication	if (is_int($permission)) {
		0 ignored issues – show Duplication introduced 2018-05-23 13:04 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
203		$permission = app(Permission::class)->findById($permission, $this->getDefaultGuardName());
204		if (! $permission) {
205		return false;
206		}
207		}
208
209		if (! $permission instanceof Permission) {
210		return false;
211		}
212
213		return $this->permissions->contains('id', $permission->id);
		0 ignored issues – show Bug introduced 2015-12-21 23:32 UTC by Report Bug Copy Issue Report The property `permissions` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history... Bug introduced 2018-08-08 00:11 UTC by Report Bug Copy Issue Report Accessing `id` on the interface `Spatie\Permission\Contracts\Permission` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
214		}
215
216		/**
217		* Return all the permissions the model has via roles.
218		*/
219		public function getPermissionsViaRoles(): Collection
220		{
221		return $this->load('roles', 'roles.permissions')
		0 ignored issues – show Bug introduced 2017-02-22 12:47 UTC by Report Bug Copy Issue Report It seems like `load()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
222		->roles->flatMap(function ($role) {
223		return $role->permissions;
224		})->sort()->values();
225		}
226
227		/**
228		* Return all the permissions the model has, both directly and via roles.
229		*/
230		public function getAllPermissions(): Collection
231		{
232		return $this->permissions
233		->merge($this->getPermissionsViaRoles())
234		->sort()
235		->values();
236		}
237
238		/**
239		* Grant the given permission(s) to a role.
240		*
241		* @param string\|array\|\Spatie\Permission\Contracts\Permission\|\Illuminate\Support\Collection $permissions
242		*
243		* @return $this
244		*/
245	View Code Duplication	public function givePermissionTo(...$permissions)
		0 ignored issues – show Duplication introduced 2017-08-07 17:04 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
246		{
247		$permissions = collect($permissions)
248		->flatten()
249		->map(function ($permission) {
250		return $this->getStoredPermission($permission);
251		})
252		->filter(function ($permission) {
253		return $permission instanceof Permission;
254		})
255		->each(function ($permission) {
256		$this->ensureModelSharesGuard($permission);
257		})
258		->all();
259
260		$this->permissions()->saveMany($permissions);
261
262		$this->forgetCachedPermissions();
263
264		return $this;
265		}
266
267		/**
268		* Remove all current permissions and set the given ones.
269		*
270		* @param string\|array\|\Spatie\Permission\Contracts\Permission\|\Illuminate\Support\Collection $permissions
271		*
272		* @return $this
273		*/
274		public function syncPermissions(...$permissions)
275		{
276		$this->permissions()->detach();
277
278		return $this->givePermissionTo($permissions);
279		}
280
281		/**
282		* Revoke the given permission.
283		*
284		* @param \Spatie\Permission\Contracts\Permission\|\Spatie\Permission\Contracts\Permission[]\|string\|string[] $permission
285		*
286		* @return $this
287		*/
288		public function revokePermissionTo($permission)
289		{
290		$this->permissions()->detach($this->getStoredPermission($permission));
291
292		$this->forgetCachedPermissions();
293
294		return $this;
295		}
296
297		/**
298		* @param string\|array\|\Spatie\Permission\Contracts\Permission\|\Illuminate\Support\Collection $permissions
299		*
300		* @return \Spatie\Permission\Contracts\Permission\|\Spatie\Permission\Contracts\Permission[]\|\Illuminate\Support\Collection
301		*/
302		protected function getStoredPermission($permissions)
303		{
304		if (is_numeric($permissions)) {
305		return app(Permission::class)->findById($permissions, $this->getDefaultGuardName());
306		}
307
308		if (is_string($permissions)) {
309		return app(Permission::class)->findByName($permissions, $this->getDefaultGuardName());
310		}
311
312		if (is_array($permissions)) {
313		return app(Permission::class)
314		->whereIn('name', $permissions)
315		->whereIn('guard_name', $this->getGuardNames())
316		->get();
317		}
318
319		return $permissions;
320		}
321
322		/**
323		* @param \Spatie\Permission\Contracts\Permission\|\Spatie\Permission\Contracts\Role $roleOrPermission
324		*
325		* @throws \Spatie\Permission\Exceptions\GuardDoesNotMatch
326		*/
327		protected function ensureModelSharesGuard($roleOrPermission)
328		{
329		if (! $this->getGuardNames()->contains($roleOrPermission->guard_name)) {
330		throw GuardDoesNotMatch::create($roleOrPermission->guard_name, $this->getGuardNames());
331		}
332		}
333
334		protected function getGuardNames(): Collection
335		{
336		return Guard::getNames($this);
337		}
338
339		protected function getDefaultGuardName(): string
340		{
341		return Guard::getDefaultName($this);
342		}
343
344		/**
345		* Forget the cached permissions.
346		*/
347		public function forgetCachedPermissions()
348		{
349		app(PermissionRegistrar::class)->forgetCachedPermissions();
350		}
351		}
352

spatie / laravel-permission

Pull Request — master (#820)

HasPermissions B

Complexity

Size/Duplication

Coupling/Cohesion

Importance

19 Methods

How to fix Duplicated Code Complexity

Duplicated Code

Complex Class

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like