Formularium\Frontend\Materialize\RenderableMaterializeInputTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	_editable()	0	24	3
A	viewable()	0	3	1

<?php

namespace Formularium\Frontend\Materialize;

use Formularium\Field;
use Formularium\HTMLElement;

trait RenderableMaterializeInputTrait
{
    use RenderableMaterializeTrait;

    public function viewable($value, Field $field, HTMLElement $previous): HTMLElement

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

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

    {
        return $previous;
    }

    public function _editable($value, Field $field, HTMLElement $previous): HTMLElement

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

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

    {
        // add extra classes
        $newContent = [];
        $input = $previous->get('input');
        $input[0]->setAttributes([
            'class' => 'validate',
        ]);
        $newContent[] = $input[0];
        $label = $previous->get('label');
        if ($label) {

The expression $label of type Formularium\HTMLElement[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $newContent[] = $label[0];
        }
        $comment = $previous->get('div.comment');
        if ($comment) {

The expression $comment of type Formularium\HTMLElement[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $comment[0]->setTag('span')->setAttributes([
                'class' => 'helper-text',
                'data-error' => "wrong",
                'data-success' => "right"
            ]);
            $newContent[] = $comment[0];
        }
        $previous->setContent($newContent);
        return $previous;
    }
}