Yard8\LaravelPermissions\Traits\HasPermissions

Complexity

Total Complexity

11

Size/Duplication

Total Lines	79
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	hasPermission()	0	10	2
A	hasAnyPermission()	0	16	4
A	hasRole()	0	14	3
A	permissions()	0	4	1
A	role()	0	4	1

<?php

namespace Yard8\LaravelPermissions\Traits;

use Illuminate\Support\Str;
use Yard8\LaravelPermissions\Models\Permission;
use Yard8\LaravelPermissions\Models\Role;

trait HasPermissions
{
    /**
     * Check if a user has a specific permission assigned to them.
     *
     * @param  string  $permission
     * @return  boolean
     */
    public function hasPermission($permission)
    {
        if ($this->hasRole(config('permissions.admin'))) {
            return true;
        }

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

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

        return in_array($permission, $permissions);
    }

    /**
     * Check if a user has any of the permissions assigned to them.
     *
     * @param  array  $permissions
     * @return  boolean
     */
    public function hasAnyPermission($permissions)
    {
        if ($this->hasRole(config('permissions.admin'))) {
            return true;
        }

        $assignedPermissions = $this->permissions->pluck('id')->toArray();

        foreach ($permissions as $permission) {
            if (in_array($permission, $assignedPermissions)) {
                return true;
            }
        }

        return false;
    }

    /**
     * Check if a user has has one of the roles assigned to them.
     *
     * @param  string|array  $roles
     * @return  boolean
     */
    public function hasRole($roles)
    {
        if (is_null($this->role)) {

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

            return false;
        }

        $role = Str::slug($this->role->name);

        if (is_array($roles)) {
            return in_array($role, $roles);
        }

        return $role === $roles;
    }

    /**
     * Get the permissions assigned to the user.
     */
    public function permissions()
    {
        return $this->belongsToMany(Permission::class);

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

    }

    /**
     * Get the role assigned to the user.
     */
    public function role()
    {
        return $this->belongsTo(Role::class);

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

    }
}