Nip_Form_Element_CheckboxGroup

Complexity

Total Complexity

12

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getDataFromRequest()	0	14	4
A	getValue()	0	13	4
A	getNewElement()	0	10	2
A	isRequestArray()	0	3	1
A	setValue()	0	3	1

<?php

/**
 * Class Nip_Form_Element_CheckboxGroup
 *
 * @method Nip_Form_Element_Checkbox[] getElements()
 */
class Nip_Form_Element_CheckboxGroup extends Nip_Form_Element_Input_Group

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    protected $_type = 'checkboxGroup';

    /**
     * @param string $requester
     * @return null
     */
    public function getValue($requester = 'abstract')

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

    {
        $elements = $this->getElements();
        $data = [];
        if ($elements) {

The expression $elements of type Nip_Form_Element_Checkbox[] 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.

            foreach ($elements as $element) {
                if ($element->isChecked()) {
                    $data[] = $element->getValue();

Are you sure the assignment to $data[] is correct as $element->getValue() targeting Nip\Form\Elements\AbstractElement::getValue() seems to always return null.

                }
            }
        }

        return $data;

The expression return $data returns the type array|array<mixed,null> which is incompatible with the documented return type null.

    }

    /** @noinspection PhpMissingParentCallCommonInspection
     * @inheritdoc
     */
    public function setValue($value)
    {
        return $this->getDataFromRequest($value);
    }

    /** @noinspection PhpMissingParentCallCommonInspection
     * @inheritdoc
     */
    public function getDataFromRequest($request)
    {
        $elements = $this->getElements();
        if (is_array($request)) {
            foreach ($elements as $key => $element) {
                $element->setChecked(in_array($key, $request));
            }
        } else {
            foreach ($elements as $key => $element) {
                $element->setChecked(false);
            }
        }

        return $this;
    }

    /**
     * @return Nip_Form_Element_Abstract
     */
    public function getNewElement()
    {
        $element = $this->getForm()->getNewElement('checkbox');
        $name = $this->getName();

Are you sure the assignment to $name is correct as $this->getName() targeting Nip\Form\Elements\AbstractElement::getName() seems to always return null.

        if (!strpos($name, '[]')) {
            $name = $name.'[]';
        }
        $element->setName($name);

        return $element;
    }

    /**
     * @return bool
     */
    public function isRequestArray()
    {
        return true;
    }
}