Nip\Records\Registry\Traits\IsCachedTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Test Coverage

Coverage

53.85%

Importance

Changes	1
Bugs	0	Features	1

4 Methods

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

<?php

namespace Nip\Records\Registry\Traits;

use Nip\Cache\Cacheable\CanCache;

/**
 * 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 $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();
        if (is_array($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?

        }
    }
}