EditableRadioField

Complexity

Total Complexity

6

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

41.18%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	8	1
A	getSelectorField()	0	6	2
A	isRadioField()	0	3	1
A	getFormField()	0	13	2

<?php
/**
 * EditableRadioField
 *
 * Represents a set of selectable radio buttons
 *
 * @package userforms
 */

class EditableRadioField extends EditableMultipleOptionField

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

{

    private static $singular_name = 'Radio Group';

Consider using a different property name as you override a private property of the parent class.

The property $singular_name is not used and could be removed.

    private static $plural_name = 'Radio Groups';

Consider using a different property name as you override a private property of the parent class.

The property $plural_name is not used and could be removed.

    /**
     * @return FieldList
     */
    public function getCMSFields()
    {
        $fields = parent::getCMSFields();

        $fields->removeByName('Default');

        return $fields;
    }

    public function getFormField()
    {
        $field = OptionsetField::create($this->Name, $this->EscapedTitle, $this->getOptionsMap());

The property EscapedTitle does not exist on object<EditableRadioField>. Since you implemented __get, maybe consider adding a @property annotation.

        $field->setFieldHolderTemplate('UserFormsMultipleOptionField_holder');

        // Set default item
        $defaultOption = $this->getDefaultOptions()->first();
        if ($defaultOption) {
            $field->setValue($defaultOption->Value);
        }
        $this->doUpdateFormField($field);
        return $field;
    }

    public function getSelectorField(EditableCustomRule $rule, $forOnLoad = false)
    {
        // We only want to trigger on load once for the radio group - hence we focus on the first option only.
        $first = $forOnLoad ? ':first' : '';
        return "$(\"input[name='{$this->Name}']{$first}\")";
    }

	public function isRadioField() {
		return true;
	}
}