App\Admin\Controllers\SettingsController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	2
Bugs	1	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	index()	0	9	2
A	writeSettings()	0	3	1
A	form()	0	11	1

<?php

namespace App\Admin\Controllers;

use App\Http\Controllers\Controller;
use Encore\Admin\Layout\Content;
use App\Models\Eloquent\OJ;
use App\Models\Eloquent\Problem;
use App\Models\Eloquent\Compiler;
use Encore\Admin\Widgets\Form;
use Encore\Admin\Widgets\Box;
use Encore\Admin\Widgets\Table;
use App\Babel\Babel;
use Arr;

class SettingsController extends Controller
{
    /**
     * Show the Testing Page.
     *
     * @return Response

The type App\Admin\Controllers\Response was not found. Did you mean Response? If so, make sure to prefix the type with \.

     */
    public function index(Content $content)
    {
        $content=$content->header(__('admin.settings.index.header'));

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

        $content=$content->description(__('admin.settings.index.description'));

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

        if (request()->isMethod('post')) {
            $this->writeSettings();
            admin_toastr(__('admin.settings.tooltip.success.message'), 'success');

It seems like __('admin.settings.tooltip.success.message') can also be of type array and array; however, parameter $message of admin_toastr() does only seem to accept string, maybe add an additional type check?

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

The expression return $content->body($this->form()) returns the type Encore\Admin\Layout\Content which is incompatible with the documented return type App\Admin\Controllers\Response.

    }

    /**
     * Make a form builder.
     *
     * @return Form
     */
    protected function form()
    {
        $box=new Box(__('admin.settings.form.header'));

It seems like __('admin.settings.form.header') can also be of type array and array; however, parameter $title of Encore\Admin\Widgets\Box::__construct() does only seem to accept string, maybe add an additional type check?

        $box->style('success');
        $form=new Form();
        $form->simplemde('terms', __('admin.settings.form.terms'))->default(setting('terms'))->help(__('admin.settings.help.terms'));
        $form->method('POST');
        $form->action(route('admin.settings.index'));
        $form->disableReset();
        $box->content($form);

$form of type Encore\Admin\Widgets\Form is incompatible with the type string expected by parameter $content of Encore\Admin\Widgets\Box::content().

        return $box;

The expression return $box returns the type Encore\Admin\Widgets\Box which is incompatible with the documented return type Encore\Admin\Widgets\Form.

    }

    protected function writeSettings()
    {
        setting(['terms' => request()->terms]);
    }
}