HasRoleAndPermission::revokePermission() - Code Metrics - Inspection of "[6.x] WIP: Allow users to directly have permission..." - yajra/laravel-acl - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#38)

by Arjay

created 2021-04-17 07:29 UTC

HasRoleAndPermission::revokePermission() A

↳ Parent: HasRoleAndPermission

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

namespace Yajra\Acl\Traits;

use Illuminate\Database\Eloquent\Relations\BelongsToMany;
use Yajra\Acl\Models\Permission;

/**
 * @mixin \Illuminate\Database\Eloquent\Model
 * @property Permission[]|\Illuminate\Database\Eloquent\Collection $permissions
 */
trait HasRoleAndPermission
{
    use HasRole {
        HasRole::getPermissions as getRolePermissions;
    }

    /**
     * Grants the given permission to the user.
     *
     * @param  \Illuminate\Support\Collection|\Illuminate\Database\Eloquent\Model|array  $id
     * @param  array  $attributes
     * @param  bool  $touch
     * @return void
     */
    public function grantPermission($id, array $attributes = [], $touch = true)
    {
        $this->permissions()->attach($id, $attributes, $touch);
    }

    /**
     * Get related permissions.
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
     */
    public function permissions(): BelongsToMany
    {
        return $this->belongsToMany(config('acl.permission', Permission::class))->withTimestamps();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
    }

    /**
     * Revokes the given permission from the user.
     *
     * @param  mixed  $ids
     * @param  bool  $touch
     * @return int
     */
    public function revokePermission($ids = null, $touch = true): int
    {
        return $this->permissions()->detach($ids, $touch);
    }

    /**
     * Syncs the given permission(s) with the user.
     *
     * @param  \Illuminate\Support\Collection|\Illuminate\Database\Eloquent\Model|array  $ids
     * @param  bool  $detaching
     * @return array
     */
    public function syncPermissions($ids, $detaching = true): array
    {
        return $this->permissions()->sync($ids, $detaching);
    }

    /**
     * Revokes all permissions from the user.
     *
     * @return int
     */
    public function revokeAllPermissions(): int
    {
        return $this->permissions()->detach();
    }

    /**
     * Get all user permissions slug.
     *
     * @return array|null
     */
    public function getPermissions(): array
    {
        $rolePermissions = $this->getRolePermissions();

        $userPermissions = $this->permissions->pluck('slug')->toArray();

        return collect($userPermissions)->merge($rolePermissions)->unique()->toArray();
    }
}


1			<?php
2
3			namespace Yajra\Acl\Traits;
4
5			use Illuminate\Database\Eloquent\Relations\BelongsToMany;
6			use Yajra\Acl\Models\Permission;
7
8			/**
9			* @mixin \Illuminate\Database\Eloquent\Model
10			* @property Permission[]\|\Illuminate\Database\Eloquent\Collection $permissions
11			*/
12			trait HasRoleAndPermission
13			{
14			use HasRole {
15			HasRole::getPermissions as getRolePermissions;
16			}
17
18			/**
19			* Grants the given permission to the user.
20			*
21			* @param \Illuminate\Support\Collection\|\Illuminate\Database\Eloquent\Model\|array $id
22			* @param array $attributes
23			* @param bool $touch
24			* @return void
25			*/
26			public function grantPermission($id, array $attributes = [], $touch = true)
27			{
28			$this->permissions()->attach($id, $attributes, $touch);
29			}
30
31			/**
32			* Get related permissions.
33			*
34			* @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
35			*/
36			public function permissions(): BelongsToMany
37			{
38			return $this->belongsToMany(config('acl.permission', Permission::class))->withTimestamps();
			0 ignored issues – show Documentation Bug introduced 2021-04-17 06:50 UTC by Report Bug Copy Issue Report The method `belongsToMany` does not exist on `object<Yajra\Acl\Traits\HasRoleAndPermission>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
39			}
40
41			/**
42			* Revokes the given permission from the user.
43			*
44			* @param mixed $ids
45			* @param bool $touch
46			* @return int
47			*/
48			public function revokePermission($ids = null, $touch = true): int
49			{
50			return $this->permissions()->detach($ids, $touch);
51			}
52
53			/**
54			* Syncs the given permission(s) with the user.
55			*
56			* @param \Illuminate\Support\Collection\|\Illuminate\Database\Eloquent\Model\|array $ids
57			* @param bool $detaching
58			* @return array
59			*/
60			public function syncPermissions($ids, $detaching = true): array
61			{
62			return $this->permissions()->sync($ids, $detaching);
63			}
64
65			/**
66			* Revokes all permissions from the user.
67			*
68			* @return int
69			*/
70			public function revokeAllPermissions(): int
71			{
72			return $this->permissions()->detach();
73			}
74
75			/**
76			* Get all user permissions slug.
77			*
78			* @return array\|null
79			*/
80			public function getPermissions(): array
81			{
82			$rolePermissions = $this->getRolePermissions();
			0 ignored issues – show Bug introduced 2021-04-17 06:50 UTC by Report Bug Copy Issue Report The method `getRolePermissions()` does not exist on `Yajra\Acl\Traits\HasRoleAndPermission`. Did you maybe mean `permissions()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
83			$userPermissions = $this->permissions->pluck('slug')->toArray();
84
85			return collect($userPermissions)->merge($rolePermissions)->unique()->toArray();
86			}
87			}
88

yajra / laravel-acl

Pull Request — master (#38)

HasRoleAndPermission::revokePermission() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like