Backpack\CRUD\app\Library\CrudPanel\Traits\Tabs

Complexity

Total Complexity

22

Size/Duplication

Total Lines	169
Duplicated Lines	0 %

Importance

Changes

0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	getTabsType()	0	3	1
A	enableHorizontalTabs()	0	3	1
A	enableVerticalTabs()	0	3	1
A	isLastTab()	0	3	1
A	getTabs()	0	16	2
A	tabsDisabled()	0	3	1
A	disableTabs()	0	5	1
A	getTabFields()	0	13	3
A	getFieldsWithoutATab()	0	9	1
A	tabsEnabled()	0	3	1
A	tabExists()	0	5	1
A	setTabsType()	0	6	1
A	disableVerticalTabs()	0	3	1
A	disableHorizontalTabs()	0	3	1
A	getLastTab()	0	9	2
A	enableTabs()	0	6	1
A	getOperationFields()	0	6	2

<?php

namespace Backpack\CRUD\app\Library\CrudPanel\Traits;

trait Tabs
{
    public function enableTabs()
    {
        $this->setOperationSetting('tabsEnabled', true);

It seems like setOperationSetting() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->setOperationSetting('tabsType', config('backpack.crud.' . $this->getCurrentOperation() . '.tabsType', 'horizontal'));

It seems like getCurrentOperation() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this->tabsEnabled();
    }

    public function disableTabs()
    {
        $this->setOperationSetting('tabsEnabled', false);

        return $this->tabsEnabled();
    }

    /**
     * @return bool
     */
    public function tabsEnabled()
    {
        return $this->getOperationSetting('tabsEnabled');

The method getOperationSetting() does not exist on Backpack\CRUD\app\Library\CrudPanel\Traits\Tabs. Did you maybe mean getOperationFields()?

    }

    /**
     * @return bool
     */
    public function tabsDisabled()
    {
        return !$this->tabsEnabled();
    }

    public function setTabsType($type)
    {
        $this->enableTabs();
        $this->setOperationSetting('tabsType', $type);

        return $this->getOperationSetting('tabsType');
    }

    /**
     * @return string
     */
    public function getTabsType()
    {
        return $this->getOperationSetting('tabsType');
    }

    public function enableVerticalTabs()
    {
        return $this->setTabsType('vertical');
    }

    public function disableVerticalTabs()
    {
        return $this->setTabsType('horizontal');
    }

    public function enableHorizontalTabs()
    {
        return $this->setTabsType('horizontal');
    }

    public function disableHorizontalTabs()
    {
        return $this->setTabsType('vertical');
    }

    /**
     * @param string $label
     *
     * @return bool
     */
    public function tabExists($label)
    {
        $tabs = $this->getTabs();

        return in_array($label, $tabs);
    }

    /**
     * @return bool|string
     */
    public function getLastTab()
    {
        $tabs = $this->getTabs();

        if (count($tabs)) {
            return last($tabs);
        }

        return false;
    }

    /**
     * @param $label
     *
     * @return bool
     */
    public function isLastTab($label)
    {
        return $this->getLastTab() == $label;
    }

    /**
     * @return \Illuminate\Support\Collection
     */
    public function getFieldsWithoutATab()
    {
        $all_fields = $this->getOperationFields();

        $fields_without_a_tab = collect($all_fields)->filter(function ($value, $key) {

The parameter $key is not used and could be removed.

            return !isset($value['tab']);
        });

        return $fields_without_a_tab;
    }

    /**
     * @param $label
     *
     * @return array|\Illuminate\Support\Collection
     */
    public function getTabFields($label)
    {
        if ($this->tabExists($label)) {
            $all_fields = $this->getOperationFields();

            $fields_for_current_tab = collect($all_fields)->filter(function ($value, $key) use ($label) {

The parameter $key is not used and could be removed.

                return isset($value['tab']) && $value['tab'] == $label;
            });

            return $fields_for_current_tab;
        }

        return [];
    }

    /**
     * @return array
     */
    public function getTabs()
    {
        $tabs = [];
        $fields = $this->getOperationFields();

        $fields_with_tabs = collect($fields)

The assignment to $fields_with_tabs is dead and can be removed.

            ->filter(function ($value, $key) {

The parameter $key is not used and could be removed.

                return isset($value['tab']);
            })
            ->each(function ($value, $key) use (&$tabs) {

The parameter $key is not used and could be removed.

                if (!in_array($value['tab'], $tabs)) {
                    $tabs[] = $value['tab'];
                }
            });

        return $tabs;
    }

    /**
     * @return mixed
     */
    public function getOperationFields()
    {
        if ($this->getCurrentOperation() == "show") {
            return $this->columns();

It seems like columns() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        } else {
            return $this->getCurrentFields();

It seems like getCurrentFields() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
}