Webfactor\Laravel\Backpack\Documents\Controllers\DocumentCrudController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	122
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	setup()	0	81	2
A	getTypeOptions()	0	8	1
A	bodyFieldType()	0	10	1
A	store()	0	8	1
A	update()	0	8	1

<?php

namespace Webfactor\Laravel\Backpack\Documents\Controllers;

use Backpack\CRUD\app\Http\Controllers\CrudController;
use Webfactor\Laravel\Backpack\Documents\Requests\DocumentRequest as StoreRequest;
use Webfactor\Laravel\Backpack\Documents\Requests\DocumentRequest as UpdateRequest;

class DocumentCrudController extends CrudController
{
    public function setup()
    {
        /*
        |--------------------------------------------------------------------------
        | BASIC CRUD INFORMATION
        |--------------------------------------------------------------------------
        */
        $this->crud->setModel(config('webfactor.documents.model_class'));
        $this->crud->setRoute(config('webfactor.documents.backend.route_prefix').'/'.config('webfactor.documents.backend.route'));

This line exceeds maximum limit of 120 characters; contains 130 characters

        $this->crud->setEntityNameStrings(trans('webfactor::documents.entity_name_singular'),
            trans('webfactor::documents.entity_name_plural'));

        /*
        |--------------------------------------------------------------------------
        | BASIC CRUD INFORMATION
        |--------------------------------------------------------------------------
        */

        // ------ CRUD FIELDS

        $this->crud->addField([
            'name'  => 'type',
            'label' => 'Dokumentenart',
            'type'  => 'select_from_array',
            'options' => $this->getTypeOptions(),
            'allows_null' => false,
            'allows_multiple' => false
        ]);

        $this->crud->addField([
            'name' => 'title',
            'label' => trans('webfactor::documents.title')
        ]);
        
        $this->crud->addField([
            'name' => 'body',
            'label' => trans('webfactor::documents.body'),
            'type' => $this->bodyFieldType()
        ]);

        // ------ CRUD COLUMNS

        $this->crud->addColumns([
            [
                'name'  => 'translated_type',
                'label' => trans('webfactor::documents.type'),
            ],
            [
                'name'  => 'title',
                'label' => trans('webfactor::documents.title')
            ],
            [
                'name'  => 'body',
                'label' => trans('webfactor::documents.body')
            ]
        ]);

        // ------ CRUD BUTTONS

        // ------ CRUD ACCESS

        collect(config('webfactor.documents.access', []))->each(function ($enabled, $key) {
            if ($enabled) {
                $this->crud->allowAccess($key);
            } else {
                $this->crud->denyAccess($key);
            }
        });

        // ------ CRUD REORDER

        // ------ CRUD DETAILS ROW

        // ------ REVISIONS

        // ------ AJAX TABLE VIEW

        // ------ DATATABLE EXPORT BUTTONS

        // ------ ADVANCED QUERIES
    }

    public function getTypeOptions()
    {
        $types = collect(config('webfactor.documents.types'));

        return $types->mapWithKeys(function ($type) {
            return [$type => trans('webfactor::documents.types.'.$type)];
        });
    }

    public function bodyFieldType()
    {
        $types = [
            'plaintext' => 'textarea',
            'html' => 'summernote',
            'md' => 'simplemde'
        ];

        return $types[config('webfactor.documents.body_type', 'plaintext')];
    }

    public function store(StoreRequest $request)
    {
        // your additional operations before save here
        $redirect_location = parent::storeCrud($request);

It seems like you call parent on a different method (storeCrud() instead of store()). Are you sure this is correct? If so, you might want to change this to $this->storeCrud().

        // 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 update(UpdateRequest $request)
    {
        // your additional operations before save here
        $redirect_location = parent::updateCrud($request);

It seems like you call parent on a different method (updateCrud() instead of update()). Are you sure this is correct? If so, you might want to change this to $this->updateCrud().

        // 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;
    }
}