GeneaLabs\LaravelModelCaching\QueryOrModelCaller

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	__call()	0	10	2

<?php

namespace GeneaLabs\LaravelModelCaching;

use Illuminate\Database\Eloquent\Builder as EloquentBuilder;
use Illuminate\Database\Eloquent\Model;

class QueryOrModelCaller
{
    /**
     * @var CachedBuilder
     */
    protected $query;
    /**
     * @var Model
     */
    protected $model;
    /**
     * @var bool
     */
    protected $disableInConfig;

    /**
     * QueryOrModelCaller constructor.
     *
     * @param EloquentBuilder $query
     * @param Model           $model
     * @param bool            $disableInConfig
     */
    public function __construct(EloquentBuilder $query, Model $model, bool $disableInConfig)
    {
        $this->query = $query;

$query is of type Illuminate\Database\Eloquent\Builder, but the property $query was declared to be of type GeneaLabs\LaravelModelCaching\CachedBuilder. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        $this->model = $model;
        $this->disableInConfig = $disableInConfig;
    }

    /**
     * @param $name
     * @param $arguments
     *
     * @return mixed
     */
    public function __call($name, $arguments)
    {
        if ($name == 'all')
            $result = call_user_func_array([$this->model, 'all'], $arguments);
        else
            $result = call_user_func_array([$this->query, $name], $arguments);

        config()->set('laravel-model-caching.disabled', $this->disableInConfig);

        return $result;
    }
}