Lagdo\UiBuilder\Component\Base\Traits\InputLabelTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	label()	0	8	2
A	setForLabelAttr()	0	5	3

<?php

namespace Lagdo\UiBuilder\Component\Base\Traits;

use Lagdo\UiBuilder\Component\HtmlElement;
use Lagdo\UiBuilder\Component\Html\Html;

trait InputLabelTrait
{
    /**
     * @var HtmlElement|null
     */
    private HtmlElement|null $label = null;

    /**
     * @param string $label
     * @param array $attributes
     * @param bool $before
     *
     * @return static
     */
    public function label(string $label, array $attributes = [], bool $before = true): static
    {
        $this->label = $before ?
            $this->addSiblingPrev('label',  $attributes) :

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

            $this->addSiblingNext('label',  $attributes);

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

        $this->label->addBaseClass('form-label');

The method addBaseClass() does not exist on null.

        $this->label->addChild(new Html($label));
        return $this;
    }

    /**
     * @return void
     */
    protected function setForLabelAttr(): void
    {
        // Set the "for" attribute on the label, if it was created.
        if ($this->label !== null && $this->element()->hasAttribute('id')) {

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

            $this->label->setAttribute('for', $this->element()->getAttribute('id'));
        }
    }
}