ModelAccessor::findByRecord() - Code Metrics - Inspection of "refactoring to v2.0.0" - flipbox/spark - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 39c748...16551a )

by Nate
created 2017-03-27 23:30 UTC
ModelAccessor::findByRecord() A

↳ Parent: ModelAccessor
Complexity

Conditions	2
Paths	2
Size

Total Lines	17
Code Lines	5
Duplication

Lines	17
Ratio	100 %
Importance

Changes
Metric	Value
dl	17
loc	17
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	2
<?php

/**
 * @package    Spark
 * @author     Flipbox Factory <[email protected]>
 * @copyright  2010-2016 Flipbox Digital Limited
 * @license    https://github.com/FlipboxFactory/Craft3-Spark/blob/master/LICENSE
 * @link       https://github.com/FlipboxFactory/Craft3-Spark
 * @since      Class available since Release 1.0.0
 */

namespace flipbox\spark\services;

use craft\helpers\Json as JsonHelper;
use flipbox\spark\exceptions\ModelNotFoundException;
use flipbox\spark\helpers\ArrayHelper;
use flipbox\spark\helpers\ModelHelper;
use flipbox\spark\models\Model;
use flipbox\spark\records\Record;
use flipbox\spark\services\traits\ModelRecordBehavior;
use yii\base\Component;
use yii\base\InvalidConfigException;
use yii\db\QueryInterface;

abstract class ModelAccessor extends Component
{

    use ModelRecordBehavior;

    /**
     * @var Model[]
     */
    protected $_cacheAll;

    /**
     * @var Model[]
     */
    protected $_cacheById = [];


    /*******************************************
     * MODEL CLASSES
     *******************************************/

    /**
     * @return string
     */
    public abstract static function modelClass(): string;


    /**
     * @return string
     */
    public static function modelClassInstance(): string
    {
        return Model::class;
    }

    /*******************************************
     * CREATE
     *******************************************/

    /**
     * @param array $config
     * @param string|null $toScenario
     * @throws InvalidConfigException
     * @return Model
     */
    public function create($config = [], string $toScenario = null): Model

    {

        // Treat records as known data and set via config
        if ($config instanceof Record) {
            return $this->createFromRecord($config, $toScenario);
        }

        // Force Array
        if (!is_array($config)) {
            $config = ArrayHelper::toArray($config, [], false);
        }

        // Set the model class
        $config['class'] = static::modelClass();

        return ModelHelper::create(
            $config,
            static::modelClassInstance(),
            $toScenario
        );

    }

    /**
     * @param Record $record
     * @param string|null $toScenario
     * @throws InvalidConfigException
     * @return Model
     */
    protected function createFromRecord(Record $record, string $toScenario = null): Model
    {

        if (null !== $toScenario) {
            $record->setScenario($toScenario);
        }

        $modelClass = static::modelClass();

        /** @var Model $model */
        $model = new $modelClass($record);

        if (null !== $toScenario) {
            $model->setScenario($toScenario);
        }

        return $model;

    }

    /*******************************************
     * FIND/GET ALL
     *******************************************/

    /**
     * @param string $toScenario
     * @return Model[]
     */
    public function findAll(string $toScenario = null)

    {

        // Check addToCache
        if (is_null($this->_cacheAll)) {

            $this->_cacheAll = [];

            // Find record in db
            if ($records = $this->findAllRecords()) {

                foreach ($records as $record) {

                    $this->_cacheAll[] = $this->findByRecord($record, $toScenario);

                }

            }

        }

        return $this->_cacheAll;

    }

    /**
     * @param string $toScenario
     * @return Model[]
     * @throws ModelNotFoundException
     */
    public function getAll(string $toScenario = null): array
    {

        if (!$models = $this->findAll($toScenario)) {

            $this->notFoundException();

        }

        return $models;

    }

    /*******************************************
     * FIND/GET
     *******************************************/

    /**
     * @param $identifier
     * @param string $toScenario
     * @return Model|null
     */
    public function find($identifier, string $toScenario = null)
    {

        if ($identifier instanceof Model) {

            $this->addToCache($identifier);

            if (null !== $toScenario) {
                $identifier->setScenario($toScenario);
            }

            return $identifier;

        } elseif ($identifier instanceof Record) {

            return $this->findByRecord($identifier, $toScenario);

        } elseif (is_numeric($identifier)) {

            return $this->findById($identifier, $toScenario);

        }

        return null;

    }

    /**
     * @param $identifier
     * @param string $toScenario
     * @return Model
     * @throws ModelNotFoundException
     */
    public function get($identifier, string $toScenario = null): Model
    {

        // Find model by ID
        if (!$model = $this->find($identifier, $toScenario)) {

            $this->notFoundException();

        }

        return $model;

    }

    /*******************************************
     * FIND/GET BY ID
     *******************************************/

    /**
     * @param int $id
     * @param string|null $toScenario
     * @return Model|null
     */
    public function findById(int $id, string $toScenario = null)

    {

        // Check cache
        if (!$model = $this->findCacheById($id)) {

            // Find record in db
            if ($record = $this->findRecordById($id)) {

                // Perhaps in cache
                $model = $this->findByRecord($record, $toScenario);

            } else {

                $this->_cacheById[$id] = null;

                return null;

            }

        }

        return $model;

    }

    /**
     * @param int $id
     * @param string|null $toScenario
     * @return Model
     * @throws ModelNotFoundException
     */
    public function getById(int $id, string $toScenario = null): Model

    {

        // Find by ID
        if (!$model = $this->findById($id, $toScenario)) {

            $this->notFoundByIdException($id);

        }

        return $model;

    }

    /**
     * @param int $id
     * @param string|null $toScenario
     * @return Model|null
     */
    public function freshFindById(int $id, string $toScenario = null)

    {

        // Find record in db
        if ($record = $this->findRecordById($id)) {

            // Create
            return $this->createFromRecord($record, $toScenario);

        }

        return null;

    }

    /**
     * @param int $id
     * @param string|null $toScenario
     * @return Model
     * @throws ModelNotFoundException
     */
    public function freshGetById(int $id, string $toScenario = null): Model

    {

        if (!$model = $this->freshFindById($id, $toScenario)) {

            $this->notFoundByIdException($id);

        }

        return $model;

    }

    /*******************************************
     * FIND/GET BY QUERY
     *******************************************/

    /**
     * @param QueryInterface $query
     * @param string $toScenario
     * @return Model[]
     */
    public function findAllByQuery(QueryInterface $query, string $toScenario = null): array

    {

        $models = array();

        foreach ($query->all() as $record) {
            $models[] = $this->findByRecord($record, $toScenario);
        }

        return $models;

    }

    /**
     * @param QueryInterface $query
     * @param string $toScenario
     * @return Model|null
     */
    public function findByQuery(QueryInterface $query, string $toScenario = null)

    {

        /** @var Record $record */
        if (!$record = $query->one()) {
            return null;
        }

        return $this->findByRecord($record, $toScenario);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

    }

    /*******************************************
     * FIND/GET BY CONDITION
     *******************************************/

    /**
     * @param $condition
     * @param string $toScenario
     * @return Model[]
     */
    public function findAllByCondition($condition, string $toScenario = null): array
    {

        $models = [];

        // Find record in db
        if ($records = $this->findAllRecordsByCondition($condition)) {

            foreach ($records as $record) {
                $models[] = $this->findByRecord($record, $toScenario);
            }

        }

        return $models;

    }

    /**
     * @param $condition
     * @param string $toScenario
     * @return Model[]
     * @throws ModelNotFoundException
     */
    public function getAllByCondition($condition, string $toScenario = null): array
    {

        if (!$models = $this->findAllByCondition($condition, $toScenario)) {

            $this->notFoundByConditionException($condition);

        }

        return $models;

    }

    /**
     * @param $condition
     * @param string $toScenario
     * @return Model|null
     */
    public function findByCondition($condition, string $toScenario = null)

    {

        // Find record in db
        if ($record = $this->findRecordByCondition($condition)) {
            return $this->findByRecord($record, $toScenario);
        }

        return null;

    }

    /**
     * @param $condition
     * @param string $toScenario
     * @return Model
     * @throws ModelNotFoundException
     */
    public function getByCondition($condition, string $toScenario = null): Model
    {

        if (!$model = $this->findByCondition($condition, $toScenario)) {

            $this->notFoundByConditionException($condition);

        }

        return $model;

    }

    /*******************************************
     * FIND/GET BY CRITERIA
     *******************************************/

    /**
     * @param $criteria
     * @param string $toScenario
     * @return Model[]
     */
    public function findAllByCriteria($criteria, string $toScenario = null): array

    {

        $models = [];

        // Find record in db
        if ($records = $this->findAllRecordsByCriteria($criteria)
        ) {

            foreach ($records as $record) {
                $models[] = $this->findByRecord($record, $toScenario);
            }

        }

        return $models;

    }

    /**
     * @param $criteria
     * @param string $toScenario
     * @return Model[]
     * @throws ModelNotFoundException
     */
    public function getAllByCriteria($criteria, string $toScenario = null): array
    {

        if (!$models = $this->findAllByCriteria($criteria, $toScenario)) {

            $this->notFoundByCriteriaException($criteria);

        }

        return $models;

    }

    /**
     * @param $criteria
     * @param string $toScenario
     * @return Model|null
     */
    public function findByCriteria($criteria, string $toScenario = null)

    {

        // Find record in db
        if ($record = $this->findRecordByCriteria($criteria)) {
            return $this->findByRecord($record, $toScenario);
        }

        return null;

    }

    /**
     * @param $criteria
     * @param string $toScenario
     * @return Model
     * @throws ModelNotFoundException
     */
    public function getByCriteria($criteria, string $toScenario = null): Model
    {

        if (!$model = $this->findByCriteria($criteria, $toScenario)) {

            $this->notFoundByCriteriaException($criteria);

        }

        return $model;

    }


    /*******************************************
     * FIND/GET BY RECORD
     *******************************************/

    /**
     * @param Record $record
     * @param string $toScenario
     * @return Model
     */
    public function findByRecord(Record $record, string $toScenario = null): Model

    {

        // Check addToCache
        if (!$model = $this->findCacheByRecord($record)) {

            // New model
            $model = $this->createFromRecord($record, $toScenario);

            // Cache it
            $this->addToCache($model);

        }

        return $model;

    }

    /**
     * @param Record $record
     * @param string $toScenario
     * @return Model
     */
    public function getByRecord(Record $record, string $toScenario = null): Model
    {
        return $this->findByRecord($record, $toScenario);
    }


    /*******************************************
     * CACHE
     *******************************************/

    /**
     * @param $identifier
     * @return Model|null
     */
    public function findCache($identifier)

    {

        if ($identifier instanceof Record) {

            return $this->findCacheByRecord($identifier);

        } elseif (is_numeric($identifier)) {

            return $this->findCacheById($identifier);

        }

        return null;

    }

    /**
     * Find an existing cache by ID
     *
     * @param $id
     * @return Model|null
     */
    public function findCacheById(int $id)
    {

        // Check if already in addToCache
        if ($this->isCachedById($id)) {

            return $this->_cacheById[$id];

        }

        return null;

    }

    /**
     * Identify whether in cache by ID
     *
     * @param $id
     * @return bool
     */
    protected function isCachedById(int $id)
    {
        return array_key_exists($id, $this->_cacheById);
    }

    /**
     * @param Model $model
     * @return $this
     */
    protected function cacheById(Model $model)
    {

        // Check if already in cache
        if (!$id = $this->isCachedById($model->id)) {

            // Cache it
            $this->_cacheById[$id] = $model;

        }

        return $this;

    }

    /**
     * @param Record $record
     * @return Model|null
     */
    public function findCacheByRecord(Record $record)
    {

        // Check if already in addToCache by id
        if ($id = $this->isCachedById($record->id)) {

            return $this->findCacheById($id);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        }

        return null;
    }

    /**
     * @param Model $model
     * @return static
     */
    public function addToCache(Model $model)
    {

        $this->cacheById($model);

        return $this;
    }


    /*******************************************
     * EXCEPTIONS
     *******************************************/

    /**
     * @throws ModelNotFoundException
     */
    protected function notFoundException()
    {

        throw new ModelNotFoundException(
            sprintf(
                "Model does not exist."
            )
        );

    }

    /**
     * @param int|null $id
     * @throws ModelNotFoundException
     */
    protected function notFoundByIdException(int $id = null)
    {

        throw new ModelNotFoundException(
            sprintf(
                'Model does not exist with the id "%s".',
                (string)$id
            )
        );

    }

    /**
     * @param null $criteria
     * @throws ModelNotFoundException
     */
    protected function notFoundByCriteriaException($criteria = null)
    {

        throw new ModelNotFoundException(
            sprintf(
                'Model does not exist with the criteria "%s".',
                (string)JsonHelper::encode($criteria)
            )
        );

    }

    /**
     * @param null $condition
     * @throws ModelNotFoundException
     */
    protected function notFoundByConditionException($condition = null)
    {

        throw new ModelNotFoundException(
            sprintf(
                'Model does not exist with the condition "%s".',
                (string)JsonHelper::encode($condition)
            )
        );

    }

}

1			<?php
2
3			/**
4			* @package Spark
5			* @author Flipbox Factory <[email protected]>
6			* @copyright 2010-2016 Flipbox Digital Limited
7			* @license https://github.com/FlipboxFactory/Craft3-Spark/blob/master/LICENSE
8			* @link https://github.com/FlipboxFactory/Craft3-Spark
9			* @since Class available since Release 1.0.0
10			*/
11
12			namespace flipbox\spark\services;
13
14			use craft\helpers\Json as JsonHelper;
15			use flipbox\spark\exceptions\ModelNotFoundException;
16			use flipbox\spark\helpers\ArrayHelper;
17			use flipbox\spark\helpers\ModelHelper;
18			use flipbox\spark\models\Model;
19			use flipbox\spark\records\Record;
20			use flipbox\spark\services\traits\ModelRecordBehavior;
21			use yii\base\Component;
22			use yii\base\InvalidConfigException;
23			use yii\db\QueryInterface;
24
25			abstract class ModelAccessor extends Component
26			{
27
28			use ModelRecordBehavior;
29
30			/**
31			* @var Model[]
32			*/
33			protected $_cacheAll;
34
35			/**
36			* @var Model[]
37			*/
38			protected $_cacheById = [];
39
40
flipbox / spark

Push — master ( 39c748...16551a )

ModelAccessor::findByRecord() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
