DropdownElement::setValueInOptGroups() - Code Metrics - Inspection of "fix: ensure that there is only 1 selected option" - splitbrain/dokuwiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — feature/OptGroups ( b0eca9...068abb )

by Michael

created 2016-12-20 12:05 UTC

DropdownElement::setValueInOptGroups() B

↳ Parent: DropdownElement

Complexity

Conditions	5
Paths	5

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
cc	5
eloc	8
c	1
b	1
f	0
nc	5
nop	1
dl	0
loc	12
rs	8.8571

<?php
namespace dokuwiki\Form;

/**
 * Class DropdownElement
 *
 * Represents a HTML select. Please note that this does not support multiple selected options!
 *
 * @package dokuwiki\Form
 */
class DropdownElement extends InputElement {

    /** @var OptGroup */
    protected $options = null;

    /** @var array OptGroup[] */
    protected $optGroups = array();

    protected $value = '';

    /**
     * @param string $name The name of this form element
     * @param string $options The available options
     * @param string $label The label text for this element (will be autoescaped)
     */
    public function __construct($name, $options, $label = '') {
        parent::__construct('dropdown', $name, $label);
        $this->rmattr('type');
        $this->options = new OptGroup(null, $options);
        $this->val('');
    }

    /**
     * @param $label
     * @param $options
     * @return OptGroup
     * @throws \Exception
     */
    public function addOptGroup($label, $options) {
        if (empty($label)) {
            throw new \InvalidArgumentException(hsc('<optgroup> must have a label!'));
        }
        $this->optGroups[] = new OptGroup($label, $options);
        return end($this->optGroups);

    }

    /**
     * Set or get the optgroups of an Dropdown-Element.
     *
     * optgroups have to be given as associative array
     *   * the key being the label of the group
     *   * the value being an array of options as defined in @see OptGroup::options()
     *
     * @param null|array $optGroups
     * @return array
     */
    public function optGroups($optGroups = null) {
        if($optGroups === null) {
            return $this->optGroups;
        }
        if (!is_array($optGroups)) {
            throw new \InvalidArgumentException(hsc('Argument must be an associative array of label => [options]!'));
        }
        $this->optGroups = array();
        foreach ($optGroups as $label => $options) {
            $this->addOptGroup($label, $options);
        }
    }

    /**
     * Get or set the options of the Dropdown
     *
     * Options can be given as associative array (value => label) or as an
     * indexd array (label = value) or as an array of arrays. In the latter
     * case an element has to look as follows:
     * option-value => array (
     *                 'label' => option-label,
     *                 'attrs' => array (
     *                                    attr-key => attr-value, ...
     *                                  )
     *                 )
     *
     * @param null|array $options
     * @return $this|array
     */
    public function options($options = null) {
        if ($options === null) {
            return $this->options->options();
        }
        $this->options = new OptGroup(null, $options);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        return $this;
    }

    /**
     * Gets or sets an attribute
     *
     * When no $value is given, the current content of the attribute is returned.
     * An empty string is returned for unset attributes.
     *
     * When a $value is given, the content is set to that value and the Element
     * itself is returned for easy chaining
     *
     * @param string $name Name of the attribute to access
     * @param null|string $value New value to set
     * @return string|$this
     */
    public function attr($name, $value = null) {
        if(strtolower($name) == 'multiple') {
            throw new \InvalidArgumentException('Sorry, the dropdown element does not support the "multiple" attribute');
        }
        return parent::attr($name, $value);
    }

    /**
     * Get or set the current value
     *
     * When setting a value that is not defined in the options, the value is ignored
     * and the first option's value is selected instead
     *
     * @param null|string $value The value to set
     * @return $this|string
     */
    public function val($value = null) {
        if($value === null) return $this->value;

        $value_exists = $this->setValueInOptGroups($value);

        if($value_exists) {
            $this->value = $value;
        } else {
            // unknown value set, select first option instead
            $this->value = $this->getFirstOption();
            $this->setValueInOptGroups($this->value);
        }

        return $this;
    }

    /**
     * Returns the first options as it will be rendered in HTML
     *
     * @return string
     */
    protected function getFirstOption() {
        $options = $this->options();
        if (!empty($options)) {
            return (string) array_shift(array_keys($options));

        }
        foreach ($this->optGroups as $optGroup) {
            $options = $optGroup->options();
            if (!empty($options)) {
                return (string) array_shift(array_keys($options));

            }
        }
    }

    /**
     * Set the value in the OptGroups, including the optgroup for the options without optgroup.
     *
     * @param string $value
     * @return bool
     */
    protected function setValueInOptGroups($value) {
        $value_exists = false;
        $isMultiSelect = $this->attributes['multiple'];
        /** @var OptGroup $optGroup */
        foreach (array_merge(array($this->options), $this->optGroups) as $optGroup) {
            $value_exists = $optGroup->setValue($value) || $value_exists;
            if ($value_exists && !$isMultiSelect) {
                $value = null;
            }
        }
        return $value_exists;
    }

    /**
     * Create the HTML for the select it self
     *
     * @return string
     */
    protected function mainElementHTML() {
        if($this->useInput) $this->prefillInput();

        $html = '<select ' . buildAttributes($this->attrs()) . '>';

        $html .= $this->options->toHTML();
        $html = array_reduce($this->optGroups, function($html, OptGroup $optGroup) {return $html . $optGroup->toHTML();}, $html);
        $html .= '</select>';

        return $html;
    }

}


1			<?php
2			namespace dokuwiki\Form;
3
4			/**
5			* Class DropdownElement
6			*
7			* Represents a HTML select. Please note that this does not support multiple selected options!
8			*
9			* @package dokuwiki\Form
10			*/
11			class DropdownElement extends InputElement {
12
13			/** @var OptGroup */
14			protected $options = null;
15
16			/** @var array OptGroup[] */
17			protected $optGroups = array();
18
19			protected $value = '';
20
21			/**
22			* @param string $name The name of this form element
23			* @param string $options The available options
24			* @param string $label The label text for this element (will be autoescaped)
25			*/
26			public function __construct($name, $options, $label = '') {
27			parent::__construct('dropdown', $name, $label);
28			$this->rmattr('type');
29			$this->options = new OptGroup(null, $options);
30			$this->val('');
31			}
32
33			/**
34			* @param $label
35			* @param $options
36			* @return OptGroup
37			* @throws \Exception
38			*/
39			public function addOptGroup($label, $options) {
40			if (empty($label)) {
41			throw new \InvalidArgumentException(hsc('<optgroup> must have a label!'));
42			}
43			$this->optGroups[] = new OptGroup($label, $options);
44			return end($this->optGroups);
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-12-20 12:09 UTC by Report Bug Copy Issue Report The expression `end($this->optGroups);` of type `dokuwiki\Form\OptGroup\|false` adds `false` to the return on line 44 which is incompatible with the return type documented by `dokuwiki\Form\DropdownElement::addOptGroup` of type `dokuwiki\Form\OptGroup`. It seems like you forgot to handle an error condition. Loading history...
45			}
46
47			/**
48			* Set or get the optgroups of an Dropdown-Element.
49			*
50			* optgroups have to be given as associative array
51			* * the key being the label of the group
52			* * the value being an array of options as defined in @see OptGroup::options()
53			*
54			* @param null\|array $optGroups
55			* @return array
56			*/
57			public function optGroups($optGroups = null) {
58			if($optGroups === null) {
59			return $this->optGroups;
60			}
61			if (!is_array($optGroups)) {
62			throw new \InvalidArgumentException(hsc('Argument must be an associative array of label => [options]!'));
63			}
64			$this->optGroups = array();
65			foreach ($optGroups as $label => $options) {
66			$this->addOptGroup($label, $options);
67			}
68			}
69
70			/**
71			* Get or set the options of the Dropdown
72			*
73			* Options can be given as associative array (value => label) or as an
74			* indexd array (label = value) or as an array of arrays. In the latter
75			* case an element has to look as follows:
76			* option-value => array (
77			* 'label' => option-label,
78			* 'attrs' => array (
79			* attr-key => attr-value, ...
80			* )
81			* )
82			*
83			* @param null\|array $options
84			* @return $this\|array
85			*/
86			public function options($options = null) {
87			if ($options === null) {
88			return $this->options->options();
89			}
90			$this->options = new OptGroup(null, $options);
			0 ignored issues – show Documentation introduced 2016-12-12 15:49 UTC by Report Bug Copy Issue Report `$options` is of type `array`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
91			return $this;
92			}
93
94			/**
95			* Gets or sets an attribute
96			*
97			* When no $value is given, the current content of the attribute is returned.
98			* An empty string is returned for unset attributes.
99			*
100			* When a $value is given, the content is set to that value and the Element
101			* itself is returned for easy chaining
102			*
103			* @param string $name Name of the attribute to access
104			* @param null\|string $value New value to set
105			* @return string\|$this
106			*/
107			public function attr($name, $value = null) {
108			if(strtolower($name) == 'multiple') {
109			throw new \InvalidArgumentException('Sorry, the dropdown element does not support the "multiple" attribute');
110			}
111			return parent::attr($name, $value);
112			}
113
114			/**
115			* Get or set the current value
116			*
117			* When setting a value that is not defined in the options, the value is ignored
118			* and the first option's value is selected instead
119			*
120			* @param null\|string $value The value to set
121			* @return $this\|string
122			*/
123			public function val($value = null) {
124			if($value === null) return $this->value;
125
126			$value_exists = $this->setValueInOptGroups($value);
127
128			if($value_exists) {
129			$this->value = $value;
130			} else {
131			// unknown value set, select first option instead
132			$this->value = $this->getFirstOption();
133			$this->setValueInOptGroups($this->value);
134			}
135
136			return $this;
137			}
138
139			/**
140			* Returns the first options as it will be rendered in HTML
141			*
142			* @return string
143			*/
144			protected function getFirstOption() {
145			$options = $this->options();
146			if (!empty($options)) {
147			return (string) array_shift(array_keys($options));
			0 ignored issues – show Bug introduced 2016-12-12 17:30 UTC by Report Bug Copy Issue Report `array_keys($options)` cannot be passed to `array_shift()` as the parameter `$array` expects a reference. Loading history...
148			}
149			foreach ($this->optGroups as $optGroup) {
150			$options = $optGroup->options();
151			if (!empty($options)) {
152			return (string) array_shift(array_keys($options));
			0 ignored issues – show Bug introduced 2016-12-12 17:30 UTC by Report Bug Copy Issue Report `array_keys($options)` cannot be passed to `array_shift()` as the parameter `$array` expects a reference. Loading history...
153			}
154			}
155			}
156
157			/**
158			* Set the value in the OptGroups, including the optgroup for the options without optgroup.
159			*
160			* @param string $value
161			* @return bool
162			*/
163			protected function setValueInOptGroups($value) {
164			$value_exists = false;
165			$isMultiSelect = $this->attributes['multiple'];
166			/** @var OptGroup $optGroup */
167			foreach (array_merge(array($this->options), $this->optGroups) as $optGroup) {
168			$value_exists = $optGroup->setValue($value) \|\| $value_exists;
169			if ($value_exists && !$isMultiSelect) {
170			$value = null;
171			}
172			}
173			return $value_exists;
174			}
175
176			/**
177			* Create the HTML for the select it self
178			*
179			* @return string
180			*/
181			protected function mainElementHTML() {
182			if($this->useInput) $this->prefillInput();
183
184			$html = '<select ' . buildAttributes($this->attrs()) . '>';
			0 ignored issues – show Bug introduced 2016-04-15 10:04 UTC by Report Bug Copy Issue Report It seems like `$this->attrs()` targeting `dokuwiki\Form\Element::attrs()` can also be of type `this<dokuwiki\Form\DropdownElement>`; however, `buildAttributes()` does only seem to accept `array`, maybe add an additional type check? This check looks at variables that are passed out again to other methods. If the outgoing method call has stricter type requirements than the method itself, an issue is raised. An additional type check may prevent trouble. Loading history...
185			$html .= $this->options->toHTML();
186			$html = array_reduce($this->optGroups, function($html, OptGroup $optGroup) {return $html . $optGroup->toHTML();}, $html);
187			$html .= '</select>';
188
189			return $html;
190			}
191
192			}
193

splitbrain / dokuwiki

Push — feature/OptGroups ( b0eca9...068abb )

DropdownElement::setValueInOptGroups() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like