Ffcms\Core\Helper\HTML\Form\MultiCheckboxField

Complexity

Total Complexity

7

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	1
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	make()	0	33	6

<?php
namespace Ffcms\Core\Helper\HTML\Form;

use Ffcms\Core\App;
use Ffcms\Core\Helper\HTML\System\NativeGenerator;
use Ffcms\Core\Helper\Type\Obj;
use Ffcms\Core\Helper\Type\Arr;
use Ffcms\Core\Helper\Type\Str;
use Ffcms\Core\Exception\SyntaxException;

class MultiCheckboxField extends NativeGenerator implements iField
{

    private $properties;

    private $name;

    private $value;

    /**
     * MultiCheckboxField constructor.
     * Pass params inside.
     * 
     * @param array $properties            
     * @param string $name            
     * @param string|null $value            
     */
    public function __construct($properties, $name, $value = null)
    {
        $this->properties = $properties;
        $this->name = $name;
        $this->value = $value;
    }

    /**
     * Build <input type="checkbox" checked {$properties} /> response
     * @todo: fix bug when option is unseted
     * {@inheritDoc}
     * @see \Ffcms\Core\Helper\HTML\Form\iField::make()
     */
    public function make()
    {
        // check if options is defined
        $options = false;

$options is not used, you could remove the assignment.

        if (isset($this->properties['options']) && Obj::isArray($this->properties['options'])) {
            $options = $this->properties['options'];
        } else {
            throw new SyntaxException('Options for field ' . self::nohtml($this->name) . ' is not defined');
        }
        
        // set field type
        $this->properties['type'] = 'checkbox';
        // set this field as array html dom object
        $this->properties['name'] .= '[]';
        unset($this->properties['value'], $this->properties['id']);
        
        $build = null;
        foreach ($options as $opt) {
            // check if this is active element
            if (Obj::isArray($this->value) && Arr::in($opt, $this->value)) {

$this->value is of type string|null, but the function expects a array.

                $this->properties['checked'] = null;
            } else {
                unset($this->properties['checked']); // remove checked if it setted before
            }
            $this->properties['value'] = $opt;
            // apply structured checkboxes style for each item
            $build .= App::$View->render('native/form/multi_checkboxes_list', [
                'item' => self::buildSingleTag('input', $this->properties) . self::nohtml($opt)
            ]);
        }
        
        return $build;
    }
}