Formularium\Frontend\Materialize\Renderable\Renderable_bool

Complexity

Total Complexity

4

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

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

<?php

namespace Formularium\Frontend\Materialize\Renderable;

use Formularium\Field;
use Formularium\Frontend\Materialize\RenderableMaterializeTrait;
use Formularium\HTMLElement;

class Renderable_bool extends \Formularium\Renderable
{
    use RenderableMaterializeTrait;
    
    public function viewable($value, Field $field, HTMLElement $previous): HTMLElement
    {
        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.

    {
        $newContent = [];
        // add extra classes
        $input = $previous->get('select');
        $input[0]->removeAttribute('class');
        $id = $input[0]->getAttribute('id')[0];
        $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];
        }
        $script = HTMLElement::factory(
            'script',
            [],
            "document.addEventListener('DOMContentLoaded', function() {
                var elems = document.querySelectorAll('#${id}');
                console.log(elems);
                var options = {};
                var instances = M.FormSelect.init(elems, options);
            });",
            true
        );
        $newContent[] = $script;
        $previous->setContent($newContent);
        return $previous;
    }
}