FormElementFileMultiple   A
last analyzed

Complexity

Total Complexity 2

Size/Duplication

Total Lines 36
Duplicated Lines 0 %

Test Coverage

Coverage 0%

Importance

Changes 0
Metric Value
eloc 14
dl 0
loc 36
ccs 0
cts 14
cp 0
rs 10
c 0
b 0
f 0
wmc 2

2 Methods

Rating   Name   Duplication   Size   Complexity  
A getHTML() 0 12 1
A __construct() 0 5 1
1
<?php
2
3
namespace Anax\HTMLForm;
4
5
/**
6
 * Form element
7
 */
8
class FormElementFileMultiple extends FormElement
9
{
10
    /**
11
     * Constructor
12
     *
13
     * @param string $name       of the element.
14
     * @param array  $attributes to set to the element. Default is an empty array.
15
     */
16
    public function __construct($name, $attributes = [])
17
    {
18
        parent::__construct($name, $attributes);
19
        $this['type'] = 'file-multiple';
20
        $this->UseNameAsDefaultLabel();
21
    }
22
23
24
25
    /**
26
     * Get HTML code for a element.
27
     *
28
     * @return string HTML code for the element.
29
     *
30
     * @SuppressWarnings(PHPMD.UnusedLocalVariable)
31
     */
32
    public function getHTML()
33
    {
34
        $details = $this->getHTMLDetails();
35
        extract($details);
0 ignored issues
show
Bug introduced by
$details of type Anax\HTMLForm\HTML is incompatible with the type array expected by parameter $var_array of extract(). ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

35
        extract(/** @scrutinizer ignore-type */ $details);
Loading history...
36
37
        return <<<EOD
0 ignored issues
show
Bug Best Practice introduced by
The expression return '<'.$wrapperEleme...'>'.$description.'</p>' returns the type string which is incompatible with the return type mandated by Anax\HTMLForm\FormElement::getHTML() of Anax\HTMLForm\HTML.

In the issue above, the returned value is violating the contract defined by the mentioned interface.

Let's take a look at an example:

interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
Loading history...
38
<{$wrapperElement}{$wrapperClass}>
39
<label for='$id'>$label</label>$brAfterLabel
40
<input id='$id' type='file' multiple{$class}{$name}{$value}{$autofocus}{$required}{$readonly}{$placeholder}{$title}{$multiple}{$pattern}{$max}{$min}{$step}/>
41
{$messages}
42
</{$wrapperElement}>
43
<p class='cf-desc'>{$description}</p>
44
EOD;
45
    }
46
}
47