UserNumericField

Complexity

Total Complexity

4

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getCMSFields()	0	10	1
A	createField()	0	14	3

<?php

/**
 * Class UserNumericField
 *
 * @property int Min
 * @property int Max
 *
 * @author Bram de Leeuw
 * @package UserTextField
 */
class UserNumericField extends Broarm\EventTickets\UserField

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

{
    /**
     * @var NumericField
     */
    protected $fieldType = 'NumericField';

    private static $db = array(

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

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

        'Min' => 'Int',
        'Max' => 'Int'
    );

    public function getCMSFields()

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

    {
        $fields = parent::getCMSFields();
        $fields->addFieldsToTab('Root.Validation', array(
            NumericField::create('Min', _t('UserNumericField.Min', 'Minimum required number')),
            NumericField::create('Max', _t('UserNumericField.Max', 'Maximum required number'))
        ));

        return $fields;
    }

    /**
     * Create a default text field
     *
     * @param string $fieldName
     * @param null   $defaultValue
     *
     * @return NumericField|FormField
     */
    public function createField($fieldName, $defaultValue = null)
    {
        $numericField = parent::createField($fieldName, $defaultValue);

        if (!empty($this->Min)) {
            $numericField->setAttribute('min', $this->Min);
        }

        if (!empty($this->Max)) {
            $numericField->setAttribute('max', $this->Max);
        }

        return $numericField;
    }
}