Nip_Form_Renderer_Elements_RadioGroup

Complexity

Total Complexity

5

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	checkDefaultValue()	0	7	4
A	generateElement()	0	5	1

<?php

/**
 * Class Nip_Form_Renderer_Elements_RadioGroup
 *
 * @method \Nip_Form_Element_RadioGroup getElement()
 */
class Nip_Form_Renderer_Elements_RadioGroup extends Nip_Form_Renderer_Elements_Input_Group
{
    /**
     * @return string
     */
    public function generateElement()
    {
        $this->checkDefaultValue();

        return parent::generateElement();
    }

    protected function checkDefaultValue()
    {
        if (!$this->getElement()->hasValue() && $this->getElement()->isAutoSelectFirst()) {
            $elements = $this->getElement()->getElements();
            if ($elements) {

The expression $elements of type array 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.

                $element = reset($elements);
                $element->setChecked(true);
            }
        }
    }
}