Sunnysideup\CMSNiceties\Traits\CMSNicetiesTraitForRightTitles

Complexity

Total Complexity

6

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	addRightTitles()	0	12	6

<?php

namespace Sunnysideup\CMSNiceties\Traits;

// use SilverStripe\Forms\GridField\GridFieldArchiveAction;

trait CMSNicetiesTraitForRightTitles
{
    public function addRightTitles($fields)
    {
        $rightFieldDescriptions = $this->Config()->get('field_labels_right');

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

        if (is_array($rightFieldDescriptions) && count($rightFieldDescriptions)) {
            foreach ($rightFieldDescriptions as $field => $desc) {
                $formField = $fields->DataFieldByName($field);
                if (! $formField) {
                    $formField = $fields->DataFieldByName($field . 'ID');
                }

                if ($formField) {
                    $formField->setDescription($desc);
                }
            }
        }

        //...
    }
}