Nip\Cache\CacheManager\HasStoresTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Test Coverage

Coverage

94.44%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	resolve()	0	14	3
A	repository()	0	3	1
A	getStore()	0	4	1
A	store()	0	4	2
A	getDefaultStore()	0	3	1

<?php

namespace Nip\Cache\CacheManager;

use InvalidArgumentException;
use Nip\Cache\Stores\Repository;

/**
 * Trait HasRepositoryTrait
 * @package Nip\Cache\Traits
 */
trait HasStoresTrait
{
    /**
     * The array of resolved cache stores.
     *
     * @var array
     */
    protected $stores = [];

    /**
     * Get a cache store instance by name.
     *
     * @param string|null $name
     * @return Repository
     */
    public function store($name = null)
    {
        $name = $name ?: $this->getDefaultStore();
        return $this->getStore($name);
    }

    /**
     * Get the default cache driver name.
     *
     * @return string
     */
    public function getDefaultStore()
    {
        return $this->getConfig('default', 'file');

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

    }

    /**
     * Attempt to get the store from the local cache.
     *
     * @param string $name
     * @return Repository
     */
    protected function getStore($name)
    {
        $this->stores[$name] = $this->stores[$name] ?? $this->resolve($name);
        return $this->stores[$name];
    }

    /**
     * @param $name
     * @return Repository
     */
    protected function resolve($name)
    {
        $config = $this->getConfigStore($name);

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

        if (is_null($config)) {
            if ($name == 'file') {
                $config = ['driver' => 'file'];
            } else {
                throw new InvalidArgumentException("Cache store [{$name}] is not defined.");
            }
        }

        return $this->repository(
            $this->createDriver($config['driver'], $config)

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

        );
    }

    /**
     * @param $driver
     * @return Repository
     */
    protected function repository($driver)
    {
        return new Repository($driver);
    }
}