GinoPane\BlogTaxonomy\Controllers\Series

Complexity

Total Complexity

6

Size/Duplication

Total Lines	71
Duplicated Lines	39.44 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	update()	12	12	2
A	onBulkDelete()	16	16	3

<?php

namespace GinoPane\BlogTaxonomy\Controllers;

use Flash;
use BackendMenu;
use Backend\Classes\Controller;
use GinoPane\BlogTaxonomy\Plugin;
use Backend\Behaviors\FormController;
use Backend\Behaviors\ListController;
use Backend\Behaviors\RelationController;
use GinoPane\BlogTaxonomy\Models\Series as SeriesModel;

/**
 * Class Series
 *
 * @package GinoPane\BlogTaxonomy\Controllers
 */
class Series extends Controller
{
    /**
     * Behaviours implemented by the controller
     *
     * @var array
     */
    public $implement = [
        FormController::class,
        ListController::class,
        RelationController::class
    ];

    public $formConfig = 'config_form.yaml';
    public $listConfig = 'config_list.yaml';
    public $relationConfig = 'config_relation.yaml';

    /**
     * Series constructor
     */
    public function __construct()
    {
        parent::__construct();

        BackendMenu::setContext(Plugin::REQUIRED_PLUGIN_RAINLAB_BLOG, 'blog', 'series');
    }

    /**
     * Controller "update" action used for updating existing model records.
     * This action takes a record identifier (primary key of the model)
     * to locate the record used for sourcing the existing form values.
     *
     * @param int $recordId Record identifier

Should the type for parameter $recordId not be integer|null?

     * @param string $context Form context

Should the type for parameter $context not be string|null?

     * @return void
     */
    public function update($recordId = null, $context = null)

This method seems to be duplicated in your project.

    {
        $series = SeriesModel::whereId($recordId)->first();

        if ($series !== null) {
            $this->pageTitle = trans(Plugin::LOCALIZATION_KEY . 'form.series.edit_title', ['series' => $series->title]);
        } else {
            $this->pageTitle = trans(Plugin::LOCALIZATION_KEY . 'form.series.series_does_not_exist');
        }

        return $this->asExtension('FormController')->update($recordId, $context);
    }

    /**
     * Remove multiple tags
     *
     * @return mixed
     */
    public function onBulkDelete()

This method seems to be duplicated in your project.

    {
        if ($checkedIds = (array)post('checked', [])) {
            $delete = SeriesModel::whereIn('id', $checkedIds)->delete();
        }

        if (empty($delete)) {
            Flash::error(e(trans(Plugin::LOCALIZATION_KEY . 'form.errors.unknown')));

            return;
        }

        Flash::success(e(trans(Plugin::LOCALIZATION_KEY . 'form.series.delete_series_success')));

        return $this->listRefresh();
    }
}