App\Admin\Controllers\DojoPassesController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	156
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	show()	0	6	1
A	index()	0	6	1
A	edit()	0	6	1
A	grid()	0	27	2
A	form()	0	35	4
A	detail()	0	12	1
A	create()	0	6	1

<?php

namespace App\Admin\Controllers;

use App\Models\Eloquent\Dojo\DojoPass;
use App\Models\Eloquent\Dojo\Dojo;
use App\Models\Eloquent\User;
use App\Http\Controllers\Controller;
use Illuminate\Support\MessageBag;
use Encore\Admin\Controllers\HasResourceActions;
use Encore\Admin\Form;
use Encore\Admin\Grid;
use Encore\Admin\Layout\Content;
use Encore\Admin\Show;

class DojoPassesController extends Controller
{
    use HasResourceActions;

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

It seems like __('admin.dojopasses.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.dojopasses.index.description'))

It seems like __('admin.dojopasses.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.dojopasses.show.header'))

It seems like __('admin.dojopasses.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.dojopasses.show.description'))

It seems like __('admin.dojopasses.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.dojopasses.edit.header'))

It seems like __('admin.dojopasses.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.dojopasses.edit.description'))

It seems like __('admin.dojopasses.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.dojopasses.create.header'))

It seems like __('admin.dojopasses.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.dojopasses.create.description'))

It seems like __('admin.dojopasses.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 DojoPass);
        $grid->column('id', "ID")->sortable();
        $grid->column("dojo_name", __('admin.dojopasses.dojo'))->display(function() {

It seems like __('admin.dojopasses.dojo') 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?

            return $this->dojo->name;

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

        });
        $grid->column("user_readable", __('admin.dojopasses.user'))->display(function() {
            return $this->user->readable_name;

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

        });
        $grid->column('updated_at', __('admin.dojopasses.updated_at'));

        $grid->filter(function(Grid\Filter $filter) {
            $filter->disableIdFilter();
            $filter->column(6, function($filter) {
                $filter->equal('dojo_id', __('admin.dojopasses.dojo'))->select(Dojo::all()->pluck('name', 'id'));
            });
            $filter->column(6, function($filter) {
                $filter->equal('user_id', __('admin.dojopasses.user'))->select(function ($id) {
                    $user = User::find($id);
                    if ($user) {
                        return [$user->id => $user->readable_name];
                    }
                })->config('minimumInputLength', 4)->ajax(route('admin.api.users'));
            });
        });
        return $grid;
    }

    /**
     * Make a show builder.
     *
     * @param mixed $id
     * @return Show
     */
    protected function detail($id)
    {
        $show=new Show(DojoPass::findOrFail($id));
        $show->field('id', "ID");
        $show->field("dojo_name", __('admin.dojopasses.dojo'))->as(function() {

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

            return $this->dojo->name;

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

        });
        $show->field("user_readable", __('admin.dojopasses.user'))->as(function() {
            return $this->user->readable_name;

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

        });
        $show->field('updated_at', __('admin.dojopasses.updated_at'));
        return $show;
    }

    /**
     * Make a form builder.
     *
     * @return Form
     */
    protected function form()
    {
        $form=new Form(new DojoPass);
        $form->tab('Basic', function(Form $form) {
            $form->display('id', 'ID');
            $form->select('dojo_id', __('admin.dojopasses.dojo'))->options(Dojo::all()->pluck('name', 'id'))->rules('required');
            $form->select('user_id', __('admin.dojopasses.user'))->options(function ($id) {
                $user = User::find($id);
                if ($user) {
                    return [$user->id => $user->readable_name];
                }
            })->config('minimumInputLength', 4)->ajax(route('admin.api.users'))->rules('required');

            $form->saving(function(Form $form) {
                $err=function($msg, $title='Error occur.') {
                    $error=new MessageBag([
                        'title'   => $title,
                        'message' => $msg,
                    ]);
                    return back()->with(compact('error'));
                };
                $user_id=$form->user_id;
                $dojo_id=$form->dojo_id;
                $pass=DojoPass::where([
                    "dojo_id" => $dojo_id,
                    "user_id" => $user_id,
                ])->first();

                $pass_id=$form->model()->id ?? null;
                if (!blank($pass) && $pass->id!=$pass_id) {
                    return $err('User has passed this dojo', 'Error occured.');
                }
            });
        });
        return $form;
    }
}