Formularium\Frontend\Materialize\RenderableMaterializeInputTrait::_editable()

Complexity

Conditions	3
Paths	4

Size

Total Lines	24
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    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;
    }