Nip\Records\Registry\Traits\IsCachedTrait::doLoad()

Complexity

Conditions	3
Paths	2

Size

Total Lines	15
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	4.125

Importance

Changes	2
Bugs	1	Features	1

<?php
    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?

        }