Sco\Admin\Config\ModelConfig

Complexity

Total Complexity

9

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getModel()	0	8	2
A	filters()	0	5	1
A	orderBy()	0	5	1
A	paginate()	0	6	1
A	parseWhere()	0	11	4

<?php

namespace Sco\Admin\Config;

use JsonSerializable;
use Illuminate\Contracts\Support\Arrayable;
use Illuminate\Contracts\Support\Jsonable;
use Sco\Admin\Exceptions\InvalidArgumentException;

class ModelConfig extends Config implements ConfigInterface, Arrayable, Jsonable, JsonSerializable

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: jsonSerialize, toArray, toJson

{

    protected $orderBy = [];
    protected $modelFilters = [];

    /**
     * Get Model
     *
     * @return \Illuminate\Database\Eloquent\Model
     * @throws \Sco\Admin\Exceptions\InvalidArgumentException
     */
    protected function getModel()
    {
        $modelName = $this->getOriginal('model');
        if (class_exists($modelName)) {
            return new $modelName();
        }
        throw new InvalidArgumentException("class {$modelName} not found");
    }

    public function filters($filters)
    {
        $this->modelFilters = $filters;
        return $this;
    }

    public function orderBy($column, $direction = 'asc')
    {
        $this->orderBy = compact('column', 'direction');
        return $this;
    }

    public function paginate()
    {
        $data = $this->parseWhere()

$data is not used, you could remove the assignment.

            ->orderBy($this->orderBy['column'], $this->orderBy['direction'])
            ->paginate(20);
    }

    protected function parseWhere()
    {
        $model = $this->getModel();
        if (!empty($this->modelFilters)) {
            foreach ($this->modelFilters as $key => $filter) {
                list($operator, $value) = is_array($filter) ? $filter : ['=', $filter];
                $model->where($key, $operator, $value);
            }
        }
        return $model;
    }
}