bytic / form

src/Renderer/Basic.php

<?php

class Nip_Form_Renderer_Basic extends Nip_Form_Renderer_Table
{
    /**
     * @inheritDoc
     */
    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();
    }
}