FormElementSelect::getHTML() - Code Metrics - Inspection of "rename files and namespace" - canax/htmlform - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( 194522...eda1b6 )

by Mikael

created 2017-08-15 08:11 UTC

FormElementSelect::getHTML() B

↳ Parent: FormElementSelect

Complexity

Conditions	3
Paths	3

Size

Total Lines	26
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	18
nc	3
nop	0
dl	0
loc	26
rs	8.8571
c	0
b	0
f	0

<?php

namespace Anax\HTMLForm;

/**
 * Form element
 */
class FormElementSelect extends FormElement
{

    /**
     * Constructor
     *
     * @param string $name       of the element.
     * @param array  $attributes to set to the element. Default is an empty array.
     *
     * @throws Exception if missing <options>
     */
    public function __construct($name, $attributes = [])
    {
        parent::__construct($name, $attributes);
        $this['type'] = 'select';
        $this->UseNameAsDefaultLabel();
        
        if (!is_array($this['options'])) {
            throw new Exception("Select needs options, did you forget to specify them when creating the element?");
        }
    }



    /**
     * Get HTML code for a element.
     *
     * @return string HTML code for the element.
     */
    public function getHTML()
    {
        $details = $this->getHTMLDetails();
        extract($details);
        
        $options = null;
        foreach ($this['options'] as $optValue => $optText) {
            $options .= "<option value='{$optValue}'"
                . (($this['value'] == $optValue)
                    ? " selected"
                    : null)
                . ">{$optText}</option>\n";
        }

        return <<<EOD
<p>
<label for='$id'>$label</label>
<br/>
<select id='$id'{$class}{$name}{$autofocus}{$required}{$readonly}{$checked}{$title}>
{$options}
</select>
{$messages}
</p>
<p class='cf-desc'>{$description}</p>
EOD;
    }
}


1			<?php
2
3			namespace Anax\HTMLForm;
4
5			/**
6			* Form element
7			*/
8			class FormElementSelect extends FormElement
9			{
10
11			/**
12			* Constructor
13			*
14			* @param string $name of the element.
15			* @param array $attributes to set to the element. Default is an empty array.
16			*
17			* @throws Exception if missing <options>
18			*/
19			public function __construct($name, $attributes = [])
20			{
21			parent::__construct($name, $attributes);
22			$this['type'] = 'select';
23			$this->UseNameAsDefaultLabel();
24
25			if (!is_array($this['options'])) {
26			throw new Exception("Select needs options, did you forget to specify them when creating the element?");
27			}
28			}
29
30
31
32			/**
33			* Get HTML code for a element.
34			*
35			* @return string HTML code for the element.
36			*/
37			public function getHTML()
38			{
39			$details = $this->getHTMLDetails();
40			extract($details);
41
42			$options = null;
43			foreach ($this['options'] as $optValue => $optText) {
44			$options .= "<option value='{$optValue}'"
45			. (($this['value'] == $optValue)
46			? " selected"
47			: null)
48			. ">{$optText}</option>\n";
49			}
50
51			return <<<EOD
52			<p>
53			<label for='$id'>$label</label>
54			<br/>
55			<select id='$id'{$class}{$name}{$autofocus}{$required}{$readonly}{$checked}{$title}>
56			{$options}
57			</select>
58			{$messages}
59			</p>
60			<p class='cf-desc'>{$description}</p>
61			EOD;
62			}
63			}
64

canax / htmlform

Push — master ( 194522...eda1b6 )

FormElementSelect::getHTML() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like