Yaro\Jarboe\Table\CrudTraits\TabsTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getTabErrorsCount()	0	11	2
A	moveDefaultTabToEnd()	0	10	2
A	getTabs()	0	13	4

<?php

namespace Yaro\Jarboe\Table\CrudTraits;

use Yaro\Jarboe\Table\Fields\AbstractField;

The type Yaro\Jarboe\Table\Fields\AbstractField was not found. Maybe you did not declare it correctly or list all dependencies?

trait TabsTrait
{
    public function getTabErrorsCount($tabTitle, $errors)
    {
        $count = 0;
        $tabs = $this->getTabs();
        $fields = $tabs[$tabTitle];

        foreach ($fields as $field) {
            $count += count($errors->get($field->name()));
        }

        return $count;
    }

    public function getTabs($skipHiddenType = null)
    {
        $tabs = [];
        foreach ($this->getFields() as $field) {

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

            if ($skipHiddenType && $field->hidden($skipHiddenType)) {
                continue;
            }
            $tabs[$field->getTab()][] = $field;
        }

        $this->moveDefaultTabToEnd($tabs);

        return $tabs;
    }

    private function moveDefaultTabToEnd(&$tabs)
    {
        $defaultTab = AbstractField::DEFAULT_TAB_IDENT;
        if (!array_key_exists($defaultTab, $tabs)) {
            return;
        }

        $defaultTabFields = $tabs[$defaultTab];
        unset($tabs[$defaultTab]);
        $tabs[$defaultTab] = $defaultTabFields;
    }
}