CheckboxList::setValue() - Code Metrics - Inspection of "Better name organization. Move element classes to..." - funivan/form - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( c47098...bd2863 )

by Shcherbak

created 2016-03-25 07:53 UTC

CheckboxList::setValue() A

↳ Parent: CheckboxList

Complexity

Conditions	4
Paths	6

Size

Total Lines	15
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	15
rs	9.2
c	1
b	0
f	0
cc	4
eloc	8
nc	6
nop	1

<?php

  namespace Fiv\Form\Element;

  /**
   * Class TextArea
   * Generate
   * ```html
   * <input type="checkbox" name="languages[]" value="en">
   * <input type="checkbox" name="languages[]" value="ru">
   * ```
   * html tags
   *
   * @author  Ivan Shcherbak <[email protected]>
   * @package Fiv\Form
   */
  class CheckboxList extends \Fiv\Form\Element\Multiple {

    /**
     * @param array|string $data
     * @return $this
     */
    public function setValue($data) {
      if (!is_array($data)) {
        $data = (array) $data;
      }

      foreach ($data as $i => $value) {
        # remove invalid values
        if (!isset($this->options[$value])) {
          unset($data[$i]);
        }
      }

      $this->value = array_values($data);

      return $this;
    }


    /**
     * Alias of setValue
     *
     * @param string[] $values
     * @return $this
     */
    public function setChecked($values) {
      return $this->setValue($values);
    }


    /**
     * @return string
     */
    public function render() {
      $html = '';
      $currentValues = $this->getValue();
      $attributes = $this->attributes;
      $attributes['type'] = 'checkbox';

      $name = $this->getName();

      foreach ($this->options as $value => $text) {
        $attributes['value'] = $value;
        $attributes['name'] = $name . '[]';
        if (in_array($value, $currentValues)) {
          $attributes['checked'] = 'checked';
        } else {
          unset($attributes['checked']);
        }

        $html .= '<label>' . static::tag('input', $attributes) . $text . '</label>';
      }

      return $html;
    }
  }

1			<?php
2
3			namespace Fiv\Form\Element;
4
5			/**
6			* Class TextArea
7			* Generate
8			* ```html
9			* <input type="checkbox" name="languages[]" value="en">
10			* <input type="checkbox" name="languages[]" value="ru">
11			* ```
12			* html tags
13			*
14			* @author Ivan Shcherbak <[email protected]>
15			* @package Fiv\Form
16			*/
17			class CheckboxList extends \Fiv\Form\Element\Multiple {
18
19			/**
20			* @param array\|string $data
21			* @return $this
22			*/
23			public function setValue($data) {
24			if (!is_array($data)) {
25			$data = (array) $data;
26			}
27
28			foreach ($data as $i => $value) {
29			# remove invalid values
30			if (!isset($this->options[$value])) {
31			unset($data[$i]);
32			}
33			}
34
35			$this->value = array_values($data);
			0 ignored issues – show Documentation Bug introduced 2016-01-18 09:56 UTC by Report Bug Copy Issue Report It seems like `array_values($data)` of type `array<integer,?>` is incompatible with the declared type `null\|string` of property `$value`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
36			return $this;
37			}
38
39
40			/**
41			* Alias of setValue
42			*
43			* @param string[] $values
44			* @return $this
45			*/
46			public function setChecked($values) {
47			return $this->setValue($values);
48			}
49
50
51			/**
52			* @return string
53			*/
54			public function render() {
55			$html = '';
56			$currentValues = $this->getValue();
57			$attributes = $this->attributes;
58			$attributes['type'] = 'checkbox';
59
60			$name = $this->getName();
61
62			foreach ($this->options as $value => $text) {
63			$attributes['value'] = $value;
64			$attributes['name'] = $name . '[]';
65			if (in_array($value, $currentValues)) {
66			$attributes['checked'] = 'checked';
67			} else {
68			unset($attributes['checked']);
69			}
70
71			$html .= '<label>' . static::tag('input', $attributes) . $text . '</label>';
72			}
73
74			return $html;
75			}
76			}

funivan / form

Push — master ( c47098...bd2863 )

CheckboxList::setValue() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like