BBSLab\NovaPermission\Traits\HasRoles

Complexity

Total Complexity

9

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getOverridePermissionCacheTags()	0	4	1
A	getOverridePermissionCacheKey()	0	8	1
A	forgetOverridePermission()	0	7	1
A	canOverridePermission()	0	20	2
A	hasPermissionToOnModel()	0	29	4

<?php

namespace BBSLab\NovaPermission\Traits;

use BBSLab\NovaPermission\Contracts\HasAuthorizations;
use Illuminate\Support\Str;
use Spatie\Permission\PermissionRegistrar;
use Spatie\Permission\Traits\HasRoles as BaseTrait;

trait HasRoles
{
    use BaseTrait;

    public function getOverridePermissionCacheTags(): array
    {
        return ['nova-permission', 'can-override'];
    }

    public function getOverridePermissionCacheKey(): string
    {
        return implode(':', [
            'nova-permission',
            'can-override',
            'user:'.$this->getKey(),

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

        ]);
    }

    public function forgetOverridePermission(): void
    {
        /** @var \Illuminate\Contracts\Cache\Factory $cacheManager */
        $cacheManager = app('cache');

        $cacheManager->flush($this->getOverridePermissionCacheKey());

The method flush() does not seem to exist on object<Illuminate\Contracts\Cache\Factory>.

    }

    public function canOverridePermission(): bool
    {
        /** @var \Illuminate\Contracts\Cache\Factory $cacheManager */
        $cacheManager = app('cache');

        $cache = method_exists($cacheManager->store(), 'tags')
            ? $cacheManager->store()->tags($this->getOverridePermissionCacheTags())

The method tags() does not seem to exist on object<Illuminate\Contracts\Cache\Repository>.

            : $cacheManager->store();

        $key = $this->getOverridePermissionCacheKey();

        return $cache->remember($key, PermissionRegistrar::$cacheExpirationTime, function () {
            $guard = config('nova.guard') ?? config('auth.defaults.guard');

            return $this->roles()
                ->where('guard_name', '=', $guard)
                ->where('override_permission', '=', true)
                ->exists();
        });
    }

    public function hasPermissionToOnModel($permission, $model = null, $guardName = null): bool
    {
        if (empty($model) || ! $model instanceof HasAuthorizations) {
            return $this->can($permission);

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

        }

        $key = implode(':', [
            'nova-permission',
            'authorization',
            class_basename($model),
            $model->getKey(),

The method getKey() does not seem to exist on object<BBSLab\NovaPermission\Contracts\HasAuthorizations>.

            Str::snake($permission),
        ]);

        /** @var \Illuminate\Contracts\Cache\Factory $cacheManager */
        $cacheManager = app('cache');

        $authorization = $cacheManager->store()
            ->remember($key, PermissionRegistrar::$cacheExpirationTime, function () use ($permission, $model, $guardName) {
                return $model->authorizations()
                    ->where('name', '=', $permission)
                    ->where('guard_name', '=', $guardName ?? $this->getDefaultGuardName())
                    ->first();
            });

        return ! empty($authorization)
            ? $this->hasPermissionTo($authorization)
            : $this->can($permission);

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

    }
}