z1haze\Acl\Traits\UserOnly

Complexity

Total Complexity

5

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	permissions()	0	4	1
A	negatePermission()	0	4	1
A	negatePermissions()	0	4	1
A	getNegatedPermissions()	0	10	1
A	cacheGetNegatedPermissions()	0	4	1

<?php

namespace z1haze\Acl\Traits;

use z1haze\Acl\Models\Permission;

trait UserOnly
{
    /**
     * USER
     * A User can have permissions explicitly assigned to it
     *
     * @return \Illuminate\Database\Eloquent\Relations\BelongsToMany
     */
    public function permissions()
    {
        return $this->belongsToMany(config('laravel-acl.permission', Permission::class))->withPivot('negated')->withTimestamps();

It seems like belongsToMany() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * USER
     * Negate a permission from a User
     *
     * @param $permission
     */
    public function negatePermission($permission)
    {
        $this->modifyPermissions([$permission], 'negate');

The method modifyPermissions() does not exist on z1haze\Acl\Traits\UserOnly. Did you maybe mean permissions()?

    }

    /**
     * Negate an array of permissions from a User
     *
     * @param $permissions
     */
    public function negatePermissions(array $permissions)
    {
        $this->modifyPermissions($permissions, 'negate');

The method modifyPermissions() does not exist on z1haze\Acl\Traits\UserOnly. Did you maybe mean permissions()?

    }

    /**
     * First try the cache to return the collection,
     * then fetch it from the database.
     *
     * See @cacheGetNegatedPermissions()
     *
     * @return \Illuminate\Support\Collection
     */
    public function getNegatedPermissions()
    {
        return \Cache::remember(
            'laravel-acl.getNegatedPermissionsForUser_' . $this->id,

The property id does not exist. Did you maybe forget to declare it?

            config('laravel-acl.cacheMinutes'),
            function() {
                return $this->cacheGetNegatedPermissions();
            }
        );
    }

    /**
     * Return a collection of permissions that
     * are negated for a User
     *
     * @return \Illuminate\Support\Collection
     */
    protected function cacheGetNegatedPermissions()
    {
        return $this->permissions()->wherePivot('negated', true)->get();
    }
}