View GitHub Repository form

Schedule Inspection
Subscribe
Completed
Push — master ( 8431a9...0bccd4 )
by Gabriel
04:28
created

Nip_Form_Renderer_Basic::renderElements()   A

↳ Parent: Nip_Form_Renderer_Basic

Complexity

Conditions 4
Paths 2

Size

Total Lines 14
Code Lines 9

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 20

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 9
c 1
b 0
f 0
dl 0
loc 14
ccs 0
cts 10
cp 0
rs 9.9666
cc 4
nc 2
nop 0
crap 20
1 
<?php
2 
class Nip_Form_Renderer_Basic extends Nip_Form_Renderer_Table
0 ignored issues
– show
Coding Style Compatibility introduced by
PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

You can fix this by adding a namespace to your class:

namespace YourVendor;

class YourClass { }

When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries.

Loading history...
3 
{
4 
    public function renderElements()
5 
    {
6 
        $elements = $this->getElements();
7 
        if ($elements) {
0 ignored issues
– show
Bug Best Practice introduced by
The expression $elements of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent.

Consider making the comparison explicit by using empty(..) or ! empty(...) instead.

Loading history...
8 
            foreach ($elements as $element) {
9 
                if (!$element->isRendered()) {
10 
                    $idRow = $element->getUniqueId();
11 
                    $this->setRowAttrib($idRow, 'class', "row " . $idRow);
12 
                    $this->addCell($idRow, 1, $element, 'label');
13 
                    $this->addCell($idRow, 2, $element, 'value');
14 
                }
15 
            }
16 
        }
17 
        return parent::renderElements();
18 
    }
19 
}
20