Nip\Form\Renderer\Traits\HasButtonRendererTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Test Coverage

Coverage

54.55%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	renderButtons()	0	14	3
A	getNewButtonRenderer()	0	10	1
A	getButtonRenderer()	0	8	2

<?php

namespace Nip\Form\Renderer\Traits;

use Nip\Form\Buttons\AbstractButton;
use Nip_Form_Renderer_Button_Abstract as AbstractButtonRenderer;

/**
 * Trait HasButtonRendererTrait
 * @package Nip\Form\Renderer\Traits
 */
trait HasButtonRendererTrait
{
    protected $buttonsRenderer = [];

    /**
     * @return string
     */
    public function renderButtons()
    {
        $return = '';
        $buttons = $this->getForm()->getButtons();

It seems like getForm() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        if ($buttons) {
            $return .= '<div class="form-actions">';
            foreach ($buttons as $button) {
                $return .= $button->render() . "\n";
            }
            $return .= '    <div class="clear"></div>';
            $return .= '</div>';
        }

        return $return;
    }

    /**
     * @param AbstractButton $button
     * @return mixed
     */
    public function getButtonRenderer(AbstractButton $button)
    {
        $name = $button->getName();
        if (!isset($this->buttonsRenderer[$name])) {
            $this->buttonsRenderer[$name] = $this->getNewButtonRenderer($button);
        }

        return $this->buttonsRenderer[$name];
    }

    /**
     * @param AbstractButton $button
     * @return mixed
     */
    protected function getNewButtonRenderer(AbstractButton $button)
    {
        $type = $button->getType();
        $name = 'Nip_Form_Renderer_Button_' . ucfirst($type);
        /** @var AbstractButtonRenderer $renderer */
        $renderer = new $name();
        $renderer->setRenderer($this);
        $renderer->setItem($button);

        return $renderer;
    }
}