Distilleries\Expendable\Http\Controllers\Admin\Base\ModelBaseController

Complexity

Total Complexity

13

Size/Duplication

Total Lines	103
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

93.75%

Importance

Changes	7
Bugs	0	Features	5

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	putDestroy()	0	14	2
A	postSearch()	0	19	3
A	generateResultSearchByIds()	0	12	3
A	getParams()	0	10	2
A	generateResultSearch()	0	20	2

<?php namespace Distilleries\Expendable\Http\Controllers\Admin\Base;

use Distilleries\Expendable\Contracts\LayoutManagerContract;
use Distilleries\Expendable\Helpers\TranslationUtils;
use Distilleries\Expendable\Models\BaseModel;
use Illuminate\Http\Request;

class ModelBaseController extends BaseController {

    /**
     * @var \Distilleries\Expendable\Models\BaseModel $model
     * Injected by the constructor
     */
    protected $model;


    // ------------------------------------------------------------------------------------------------

    public function __construct(BaseModel $model, LayoutManagerContract $layoutManager)
    {
        parent::__construct($layoutManager);
        $this->model = $model;
    }

    // ------------------------------------------------------------------------------------------------
    // ------------------------------------------------------------------------------------------------
    // ------------------------------------------------------------------------------------------------

    public function putDestroy(Request $request)
    {
        $validation = \Validator::make($request->all(), [
            'id' => 'required'
        ]);
        if ($validation->fails()) {
            return redirect()->back()->withErrors($validation)->withInput($request->all());
        }

        $data = $this->model->where($this->model->getKeyName(), $request->get('id'))->get()->last();

The method where does not exist on object<Distilleries\Expendable\Models\BaseModel>? Since you implemented __call, maybe consider adding a @method annotation.

        $data->delete();

        return redirect()->to(action('\\' . get_class($this) . '@getIndex'));
    }

    // ------------------------------------------------------------------------------------------------
    public function postSearch(Request $request, $query = null)
    {

        $ids            = $request->get('ids');
        $local_override = $this->getParams($request, 'local_override', null);

        TranslationUtils::overrideLocal($local_override);

        if (empty($query)) {
            $query = $this->model;
        }

        if (!empty($ids)) {
            return response()->json($this->generateResultSearchByIds($request, $query, $ids));
        }

        return response()->json($this->generateResultSearch($request, $query));

    }

    protected function generateResultSearchByIds(Request $request, $query, $ids)
    {
        $no_edit = $request->get('no_edit');

        if (!empty($no_edit) && method_exists($this->model, 'withoutTranslation')) {
            $query = $query->withoutTranslation();
        }

        $data = $query->whereIn($this->model->getKeyName(), $ids)->get();

        return $data;
    }

    protected function generateResultSearch(Request $request, $query)
    {
        $term  = $request->get('term');
        $page  = $this->getParams($request, 'page', 1);
        $paged = $this->getParams($request, 'page_limit', 10);

        if (empty($term)) {
            $elements = array();
            $total    = 0;
        } else {
            $elements = $query->search($term)->take($paged)->skip(($page - 1) * $paged)->get();
            $total    = $query->search($term)->count();

        }

        return [
            'total'    => $total,
            'elements' => $elements
        ];
    }

    protected function getParams(Request $request, $key, $default_value)
    {
        $element = $request->get($key);

        if (empty($element)) {
            $element = $default_value;
        }

        return $element;
    }

}