Nip_Form_DisplayGroup

Complexity

Total Complexity

9

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Test Coverage

Coverage

34.78%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	getLegend()	0	3	1
A	getForm()	0	3	1
A	render()	0	3	1
A	getRenderer()	0	7	2
A	setLegend()	0	3	1
A	setForm()	0	5	1
A	addElement()	0	5	1
A	getNewRenderer()	0	8	1

<?php

use Nip\Collections\Collection;
use Nip\Form\AbstractForm;
use Nip\Form\Elements\AbstractElement;

/**
 * Class Nip_Form_DisplayGroup
 */
class Nip_Form_DisplayGroup extends Collection
{
    use \Nip\Form\Utility\HasAttributesTrait;

    /**
     * @var Nip_Form
     */
    protected $_form;

    protected $renderer;

    /**
     * @return Nip_Form|null
     */
    public function getForm()
    {
        return $this->_form;
    }

    /**
     * @param  AbstractForm $form
     * @return Nip_Form_DisplayGroup
     */
    public function setForm(AbstractForm $form)
    {
        $this->_form = $form;

$form is of type Nip\Form\AbstractForm, but the property $_form was declared to be of type Nip_Form. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

        return $this;
    }

    /**
     * @param AbstractElement $element
     * @return $this
     */
    public function addElement(AbstractElement $element)
    {
        $this[$element->getUniqueId()] = $element;

        return $this;
    }

    /**
     * @param string $legend
     * @return Nip_Form_DisplayGroup
     */
    public function setLegend($legend)
    {
        return $this->setAttrib('legend', (string)$legend);
    }

    /**
     * @return mixed|null
     */
    public function getLegend()
    {
        return $this->getAttrib('legend');
    }

    /**
     * @return mixed
     */
    public function render()
    {
        return $this->getRenderer()->render();
    }

    /**
     * @return Nip_Form_Renderer_DisplayGroup
     */
    public function getRenderer()
    {
        if (!$this->renderer) {
            $this->renderer = $this->getNewRenderer();
        }

        return $this->renderer;
    }

    /**
     * @param string $type
     * @return Nip_Form_Renderer_DisplayGroup
     */
    public function getNewRenderer($type = 'basic')

The parameter $type is not used and could be removed.

    {
        $name = 'Nip_Form_Renderer_DisplayGroup';
        /** @var Nip_Form_Renderer_DisplayGroup $renderer */
        $renderer = new $name();
        $renderer->setGroup($this);

        return $renderer;
    }
}