App\Admin\Controllers\JudgeServerController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	149
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	detail()	0	26	3
A	form()	0	13	1
A	index()	0	6	1
A	create()	0	6	1
A	edit()	0	6	1
A	show()	0	6	1
A	grid()	0	28	3

<?php

namespace App\Admin\Controllers;

use App\Models\Eloquent\JudgeServer;
use App\Models\JudgerModel;
use App\Models\Eloquent\OJ;
use Encore\Admin\Controllers\AdminController;
use Encore\Admin\Controllers\HasResourceActions;
use Encore\Admin\Form;
use Encore\Admin\Grid;
use Encore\Admin\Layout\Content;
use Encore\Admin\Show;
use Illuminate\Database\Eloquent\Model;

class JudgeServerController extends AdminController
{
    use HasResourceActions;

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

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

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

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

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

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

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

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

It seems like __('admin.judgeservers.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 JudgeServer());

        $grid->column('jsid', 'JSID');
        $grid->column('scode', __('admin.judgeservers.scode'));

It seems like __('admin.judgeservers.scode') 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('name', __('admin.judgeservers.name'));
        $grid->column('host', __('admin.judgeservers.host'));
        $grid->column('port', __('admin.judgeservers.port'));
        $grid->column('token', __('admin.judgeservers.token'));
        $grid->column('available', __('admin.judgeservers.availability'))->display(function($available) {
            return $available ?__('admin.judgeservers.available') : __('admin.judgeservers.unavailable');
        });
        $grid->column('OJ', __('admin.judgeservers.oj'))->display(function() {
            return $this->oj->name;

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

        });
        $grid->column('usage', __('admin.judgeservers.usage'))->display(function($usage) {
            return blank($usage) ? "-" : "$usage%";
        });
        $grid->column('status', __('admin.judgeservers.status'))->display(function($status) {
            $status=JudgerModel::$status[$status];
            return '<i class="MDI '.$status['icon'].' '.$status['color'].'"></i> '.$status['text'];
        });
        $grid->column('status_update_at', __('admin.judgeservers.status_update_at'));
        $grid->column('created_at', __('admin.created_at'));
        $grid->column('updated_at', __('admin.updated_at'));

        return $grid;
    }

    /**
     * Make a show builder.
     *
     * @param mixed $id
     * @return Show
     */
    protected function detail($id)
    {
        $show=new Show(JudgeServer::findOrFail($id));

        $show->field('jsid', 'JSID');
        $show->field('scode', __('admin.judgeservers.scode'));

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

        $show->field('name', __('admin.judgeservers.name'));
        $show->field('host', __('admin.judgeservers.host'));
        $show->field('port', __('admin.judgeservers.port'));
        $show->field('token', __('admin.judgeservers.token'));
        $show->field('available', __('admin.judgeservers.availability'))->as(function($available) {
            return $available ?__('admin.judgeservers.available') : __('admin.judgeservers.unavailable');
        });
        $show->field('oj.name', __('admin.judgeservers.oj'));
        $show->field('usage', __('admin.judgeservers.usage'))->as(function($usage) {
            return blank($usage) ? "-" : "$usage%";
        });
        $show->field('status', __('admin.judgeservers.status'))->unescape()->as(function($status) {
            $status=JudgerModel::$status[$status];
            return '<i class="MDI '.$status['icon'].' '.$status['color'].'"></i> '.$status['text'];
        });
        $show->field('status_update_at', __('admin.judgeservers.status_update_at'));
        $show->field('created_at', __('admin.created_at'));
        $show->field('updated_at', __('admin.updated_at'));

        return $show;
    }

    /**
     * Make a form builder.
     *
     * @return Form
     */
    protected function form()
    {
        $form=new Form(new JudgeServer());

        $form->text('scode', __('admin.judgeservers.scode'))->required();
        $form->text('name', __('admin.judgeservers.name'))->required();
        $form->text('host', __('admin.judgeservers.host'))->required();
        $form->text('port', __('admin.judgeservers.port'))->required();
        $form->text('token', __('admin.judgeservers.token'))->required();
        $form->switch('available', __('admin.judgeservers.availability'));
        $form->select('oid', __('admin.judgeservers.oj'))->options(OJ::all()->pluck('name', 'oid'))->help(__('admin.judgeservers.help.onlinejudge'))->required();

It seems like __('admin.judgeservers.help.onlinejudge') can also be of type array and array; however, parameter $text of Encore\Admin\Form\Field::help() does only seem to accept string, maybe add an additional type check?

        $form->hidden('status', __('admin.judgeservers.status'))->default(0);
        return $form;
    }
}