Nip\Records\Registry\Traits\IsCachedTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

40.91%

Importance

Changes	2
Bugs	1	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__destruct()	0	3	1
A	offsetSet()	0	4	1
A	generateCacheData()	0	7	1
A	doLoad()	0	15	3

<?php

namespace Nip\Records\Registry\Traits;

use Nip\Cache\Cacheable\CanCache;
use Nip\Records\Instantiator\Instantiator;

/**
 * Trait IsCachedTrait
 * @package Nip\Records\Registry\Traits
 */
trait IsCachedTrait
{
    use CanCache;

    /**
     * @inheritDoc
     */
    public function offsetSet($key, $value)
    {
        $this->needsCaching(true);
        parent::offsetSet($key, $value);
    }

    public function __destruct()
    {
        $this->checkSaveCache();
    }

    /**
     * @return array
     */
    protected function generateCacheData()
    {
        return array_map(
            function ($manager) {
                return get_class($manager);
            },
            $this->all()

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

        );
    }

    protected function doLoad(): void
    {
        $data = $this->getDataFromCache();

        $instantiator = new Instantiator();
        $instantiator->setModelRegistry($this);

        if (is_array($data) && count($data)) {
            $data = array_map(
                function ($manager) use ($instantiator){
                    return $instantiator->instantiate($manager);
                },
                $data
            );
            $this->setItems($data);

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

        }
    }
}