Nip_Form_Renderer_Basic::renderElements()

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

<?php
    public function renderElements()
    {
        $elements = $this->getElements();
        if ($elements) {

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.

            foreach ($elements as $element) {
                if (!$element->isRendered()) {
                    $idRow = $element->getUniqueId();
                    $this->setRowAttrib($idRow, 'class', "row " . $idRow);
                    $this->addCell($idRow, 1, $element, 'label');
                    $this->addCell($idRow, 2, $element, 'value');
                }
            }
        }
        return parent::renderElements();
    }