UserOptionSetField

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	14	2
A	createField()	0	4	1
A	getValue()	0	4	1

<?php
use Broarm\EventTickets\UserOptionSetFieldGridFieldConfig;

/**
 * Class UserOptionSetField
 *
 * @author Bram de Leeuw
 * @package UserTextField
 *
 * @method \HasManyList Options()
 */
class UserOptionSetField extends Broarm\EventTickets\UserField

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

{
    private static $has_many = array(

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

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

        'Options' => 'Broarm\EventTickets\UserFieldOption'
    );

    public function getCMSFields()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $fields = parent::getCMSFields();
        if ($this->exists()) {
            $fields->addFieldToTab('Root.Main', GridField::create(
                'Options',
                _t('AttendeeExtraField.Options', 'Add field options'),
                $this->Options(),
                UserOptionSetFieldGridFieldConfig::create()
            ));
        }

        return $fields;
    }

    /**
     * @param string $fieldName
     * @param null   $defaultValue
     *
     * @return OptionsetField
     */
    public function createField($fieldName, $defaultValue = null)
    {
        return OptionsetField::create($fieldName, $this->Title, $this->Options()->map()->toArray(), $defaultValue);
    }

    /**
     * Get the value by set option
     *
     * @return mixed
     */
    public function getValue()
    {
        return $this->Options()->byID($this->getField('Value'))->Title;
    }
}