App\Admin\Controllers\DojoController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	141
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	form()	0	17	1
A	index()	0	6	1
A	detail()	0	10	1
A	create()	0	6	1
A	show()	0	6	1
A	grid()	0	32	2
A	edit()	0	6	1

<?php

namespace App\Admin\Controllers;

use App\Models\Eloquent\Dojo\Dojo;
use App\Http\Controllers\Controller;
use App\Models\Eloquent\Dojo\DojoPhase;
use App\Models\Eloquent\Problem;
use Encore\Admin\Controllers\HasResourceActions;
use Encore\Admin\Form;
use Encore\Admin\Grid;
use Encore\Admin\Layout\Content;
use Encore\Admin\Show;

class DojoController extends Controller
{
    use HasResourceActions;

    /**
     * Index interface.
     *
     * @param Content $content
     * @return Content
     */
    public function index(Content $content)
    {
        return $content
            ->header(__('admin.dojos.index.header'))

It seems like __('admin.dojos.index.header') can also be of type array and array; however, parameter $header of Encore\Admin\Layout\Content::header() does only seem to accept string, maybe add an additional type check?

            ->description(__('admin.dojos.index.description'))

It seems like __('admin.dojos.index.description') can also be of type array and array; however, parameter $description of Encore\Admin\Layout\Content::description() does only seem to accept string, maybe add an additional type check?

            ->body($this->grid()->render());
    }

    /**
     * Show interface.
     *
     * @param mixed $id
     * @param Content $content
     * @return Content
     */
    public function show($id, Content $content)
    {
        return $content
            ->header(__('admin.dojos.show.header'))

It seems like __('admin.dojos.show.header') can also be of type array and array; however, parameter $header of Encore\Admin\Layout\Content::header() does only seem to accept string, maybe add an additional type check?

            ->description(__('admin.dojos.show.description'))

It seems like __('admin.dojos.show.description') can also be of type array and array; however, parameter $description of Encore\Admin\Layout\Content::description() does only seem to accept string, maybe add an additional type check?

            ->body($this->detail($id));
    }

    /**
     * Edit interface.
     *
     * @param mixed $id
     * @param Content $content
     * @return Content
     */
    public function edit($id, Content $content)
    {
        return $content
            ->header(__('admin.dojos.edit.header'))

It seems like __('admin.dojos.edit.header') can also be of type array and array; however, parameter $header of Encore\Admin\Layout\Content::header() does only seem to accept string, maybe add an additional type check?

            ->description(__('admin.dojos.edit.description'))

It seems like __('admin.dojos.edit.description') can also be of type array and array; however, parameter $description of Encore\Admin\Layout\Content::description() does only seem to accept string, maybe add an additional type check?

            ->body($this->form()->edit($id));
    }

    /**
     * Create interface.
     *
     * @param Content $content
     * @return Content
     */
    public function create(Content $content)
    {
        return $content
            ->header(__('admin.dojos.create.header'))

It seems like __('admin.dojos.create.header') can also be of type array and array; however, parameter $header of Encore\Admin\Layout\Content::header() does only seem to accept string, maybe add an additional type check?

            ->description(__('admin.dojos.create.description'))

It seems like __('admin.dojos.create.description') can also be of type array and array; however, parameter $description of Encore\Admin\Layout\Content::description() does only seem to accept string, maybe add an additional type check?

            ->body($this->form());
    }

    /**
     * Make a grid builder.
     *
     * @return Grid
     */
    protected function grid()
    {
        $grid=new Grid(new Dojo);
        $grid->column('id', "ID")->sortable();
        $grid->column("name", __('admin.dojos.name'))->editable();

It seems like __('admin.dojos.name') can also be of type array and array; however, parameter $label of Encore\Admin\Grid::column() does only seem to accept string, maybe add an additional type check?

        $grid->column("dojo_phase", __('admin.dojos.phase'))->display(function() {
            return $this->phase->name;

The property phase does not exist on App\Admin\Controllers\DojoController. Did you maybe forget to declare it?

        });
        $grid->column("totproblem", __('admin.dojos.totproblem'))->display(function() {
            return $this->tot_problem;

The property tot_problem does not exist on App\Admin\Controllers\DojoController. Did you maybe forget to declare it?

        });
        $grid->column("passline", __('admin.dojos.passline'));
        $grid->column("precondition", __('admin.dojos.precondition'))->display(function($precondition) {
            $output='';
            foreach ($precondition as $p) {
                $output.='<span class="label label-primary">'.Dojo::find($p)->name.'</span> ';

The property name does not seem to exist on App\Models\Eloquent\Dojo\Dojo. Are you sure there is no database migration missing?

            }
            return $output;
        });
        $grid->column("order", __('admin.dojos.order'))->sortable();
        $grid->created_at(__('admin.created_at'));
        $grid->updated_at(__('admin.updated_at'));

        $grid->filter(function(Grid\Filter $filter) {
            $filter->column(6, function($filter) {
                $filter->like('name', __('admin.dojos.name'));
            });
            $filter->column(6, function($filter) {
                $filter->equal('dojo_phase_id', __('admin.dojos.phase'))->select(DojoPhase::all()->pluck('name', 'cid'));
            });
        });
        return $grid;
    }

    /**
     * Make a show builder.
     *
     * @param mixed $id
     * @return Show
     */
    protected function detail($id)
    {
        $show=new Show(Dojo::findOrFail($id));
        $show->id('ID');
        $show->name(__('admin.dojos.name'));
        $show->description(__('admin.dojos.description'));
        $show->dojo_phase_id(__('admin.dojos.phase'));
        $show->passline(__('admin.dojos.passline'));
        $show->order(__('admin.dojos.order'));
        return $show;
    }

    /**
     * Make a form builder.
     *
     * @return Form
     */
    protected function form()
    {
        $form=new Form(new Dojo);
        $form->tab('Basic', function(Form $form) {
            $form->display('id', 'ID');
            $form->text('name', __('admin.dojos.name'))->rules('required');
            $form->textarea('description', __('admin.dojos.description'))->rules('required');
            $form->select('dojo_phase_id', __('admin.dojos.phase'))->options(DojoPhase::all()->pluck('name', 'id'))->rules('required');
            $form->number('passline', __('admin.dojos.passline'))->default(0)->rules('required');
            $form->number('order', __('admin.dojos.order'))->default(0)->rules('required');
            $form->multipleSelect('precondition', __('admin.dojos.precondition'))->options(Dojo::all()->pluck('name', 'id'));
            $form->hasMany('problems', __('admin.dojos.problems'), function(Form\NestedForm $form) {
                $form->select('problem_id', __('admin.dojos.problem'))->options(Problem::all()->pluck('readable_name', 'pid'))->required();
                $form->number('order', __('admin.dojos.problemorder'))->default(0)->required();
            });
        });
        return $form;
    }
}