Webfactor\Laravel\Backpack\NestedModels\Controllers\NestedModelsCrudController

Complexity

Total Complexity

14

Size/Duplication

Total Lines	104
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	setup()	0	9	1
A	index()	0	12	1
A	create()	0	18	3
A	ajaxCreate()	0	13	1
A	storeCrud()	0	12	2
B	ajaxStore()	0	19	5
A	saveReorder()	0	10	1

<?php

namespace Webfactor\Laravel\Backpack\NestedModels\Controllers;

use Backpack\CRUD\app\Http\Controllers\CrudController;
use Backpack\CRUD\app\Http\Requests\CrudRequest as StoreRequest;
use Illuminate\Http\Request;

class NestedModelsCrudController extends CrudController
{

    public function setup()
    {
        parent::setup();

        $this->crud->setListView('nestedmodels::treecrud');
        $this->crud->orderBy('lft');

        $this->crud->allowAccess('reorder');
    }

    public function index()
    {
        $this->crud->hasAccessOrFail('list');

        $this->data['crud'] = $this->crud;
        $this->data['title'] = ucfirst($this->crud->entity_name_plural);

        $entries = $this->data['crud']->getEntries();
        $this->data['entries'] = $this->crud->model::loadTree($entries);

The method loadTree cannot be called on $this->crud->model (of type string).

        return view($this->crud->getListView(), $this->data);
    }

    public function create()
    {
        $request = \Request::instance();

        if ($parent = $request->get('parent')) {
            $this->crud->addField([
                'name'  => 'parent_id',
                'type'  => 'hidden',
                'value' => $parent
            ]);
        }

        if ($request->wantsJson()) {
            return $this->ajaxCreate($request);
        }

        return parent::create();
    }

    public function ajaxCreate(Request $request)

The parameter $request is not used and could be removed.

    {
        $this->crud->hasAccessOrFail('create');

        // prepare the fields you need to show
        $this->data['crud'] = $this->crud;
        $this->data['saveAction'] = $this->getSaveAction();
        $this->data['fields'] = $this->crud->getCreateFields();
        $this->data['title'] = trans('backpack::crud.add').' '.$this->crud->entity_name;

        // load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
        return view('nestedmodels::ajax.create', $this->data);
    }

    public function storeCrud(StoreRequest $request = null)

The return type could not be reliably inferred; please add a @return annotation.

    {
        if ($request->wantsJson()) {

It seems like $request is not always an object, but can also be of type null. Maybe add an additional type check?

            return $this->ajaxStore($request);

It seems like $request defined by parameter $request on line 68 can be null; however, Webfactor\Laravel\Backpack\NestedModels\Controllers\NestedModelsCrudController::ajaxStore() does not accept null, maybe add an additional type check?

        }

        // your additional operations before save here
        $redirect_location = parent::storeCrud($request);
        // your additional operations after save here
        // use $this->data['entry'] or $this->crud->entry

59% of this comment could be valid code. Did you maybe forget this after debugging?

        return $redirect_location;
    }

    public function ajaxStore(StoreRequest $request)

The return type could not be reliably inferred; please add a @return annotation.

    {
        $this->crud->hasAccessOrFail('create');

        // fallback to global request instance
        if (is_null($request)) {
            $request = \Request::instance();
        }

        // replace empty values with NULL, so that it will work with MySQL strict mode on
        foreach ($request->input() as $key => $value) {

The expression $request->input() of type string|array is not guaranteed to be traversable. How about adding an additional type check?

            if (empty($value) && $value !== '0') {
                $request->request->set($key, null);
            }
        }

        // insert item in the db
        return $this->crud->create($request->except(['save_action', '_token', '_method']));
    }

    public function saveReorder()
    {
        $this->crud->hasAccess('reorder');
        $request = \Request::instance();

        $this->crud->model::rebuildTree($request->all());

The method rebuildTree cannot be called on $this->crud->model (of type string).

        return (string) true;

    }

}