ByTIC\DocumentGenerator\PdfLetters\Forms\Fields\AdminPdfLetterFieldsFormTrait

Complexity

Total Complexity

9

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	1	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	initTextTransformElement()	0	5	2
A	initGenericElements()	0	7	1
A	initColorElement()	0	3	1
A	initAlignElement()	0	5	2
A	initSizeElement()	0	5	2
A	initPositionElements()	0	4	1

<?php

namespace ByTIC\DocumentGenerator\PdfLetters\Forms\Fields;

use ByTIC\DocumentGenerator\PdfLetters\Models\Fields\Attributes\TextTransform;

/**
 * Trait AdminPdfLetterFieldsControllerTrait
 * @package ByTIC\DocumentGenerator\PdfLetters\Forms\Fields
 */
trait AdminPdfLetterFieldsFormTrait
{
    protected function initGenericElements()
    {
        $this->initPositionElements();
        $this->initSizeElement();
        $this->initColorElement();
        $this->initAlignElement();
        $this->initTextTransformElement();
    }

    protected function initPositionElements()
    {
        $this->addInput('x', 'X', true);

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

        $this->addInput('y', 'Y', true);
    }

    protected function initColorElement()
    {
        $this->addInput('color', translator()->trans('color'), false);
    }

    protected function initSizeElement()
    {
        $this->addSelect('size', translator()->trans('size'), true);

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

        foreach (range(9, 500) as $size) {
            $this->size->addOption($size, $size);
        }
    }

    protected function initAlignElement()
    {
        $this->addSelect('align', translator()->trans('align'), true);
        foreach (['left', 'center', 'right'] as $option) {
            $this->getElement('align')->addOption($option, translator()->trans($option));

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

        }
    }

    protected function initTextTransformElement()
    {
        $this->addSelect(TextTransform::NAME, translator()->trans('text-transform'), true);
        foreach (TextTransform::OPTIONS as $option) {
            $this->getElement(TextTransform::NAME)->addOption($option, translator()->trans($option));
        }
    }
}