Helldar\Roles\Traits\Cacheable

Complexity

Total Complexity

6

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	cacheTtl()	0	3	1
A	cache()	0	10	2
A	cacheUserKey()	0	4	1
A	cacheKey()	0	6	1
A	allowCache()	0	3	1

<?php

namespace Helldar\Roles\Traits;

use Closure;
use Helldar\Roles\Facades\Config;
use Illuminate\Support\Arr;
use Illuminate\Support\Facades\Cache;
use Illuminate\Support\Str;

/** @mixin \Illuminate\Database\Eloquent\Model */
trait Cacheable
{
    protected function cache(string $prefix, Closure $callback, $values = [])
    {
        if ($this->allowCache()) {
            $key = $this->cacheKey($prefix, $values);
            $ttl = $this->cacheTtl();

            return Cache::remember($key, $ttl, $callback);
        }

        return $callback();
    }

    protected function cacheKey(string $prefix, $values = []): string
    {
        $values = Arr::flatten(Arr::wrap($values));

        return Str::slug(
            $prefix . '-' . $this->cacheUserKey() . '-' . implode('-', $values)
        );
    }

    protected function cacheUserKey(): string
    {
        return $this->getAttribute(

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

            $this->getKeyName()

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

        );
    }

    protected function cacheTtl(): int
    {
        return Config::cacheTtl();
    }

    protected function allowCache(): bool
    {
        return Config::useCache();
    }
}