Nip\Form\Elements\Traits\HasDecoratorsTrait

Complexity

Total Complexity

14

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Test Coverage

Coverage

64.29%

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	addDecorator()	0	5	1
A	attachDecorator()	0	10	2
A	newDecorator()	0	3	1
A	getDecorator()	0	13	4
A	removeDecorator()	0	15	4
A	getDecoratorsByPosition()	0	3	2

<?php

namespace Nip\Form\Elements\Traits;

use Nip\Form\Decorator\DecoratorFactory;
use Nip\Form\Decorator\Elements\AbstractDecorator;

/**
 * Trait HasDecoratorsTrait
 * @package Nip\Form\Elements\Traits
 */
trait HasDecoratorsTrait
{
    protected $decorators;

    /**
     * @param string $type
     * @return mixed
     */
    public function newDecorator($type = '')
    {
        return DecoratorFactory::createForElement($this, $type);
    }

    /**
     * @param string $type
     * @param string $position
     * @param bool|string $name
     * @return mixed
     */
    public function addDecorator($type = '', $position = 'element', $name = false)
    {
        $decorator = $this->newDecorator($type);
        $this->attachDecorator($decorator, $position, $name);

It seems like $name can also be of type string; however, parameter $name of Nip\Form\Elements\Traits\HasDecoratorsTrait::attachDecorator() does only seem to accept boolean, maybe add an additional type check?

        return $decorator;
    }

    /**
     * @param AbstractDecorator $decorator
     * @param string $position
     * @param bool $name
     * @return $this
     */
    public function attachDecorator(
        AbstractDecorator $decorator,
        $position = 'element',
        $name = false
    ) {
        $decorator->setElement($this);
        $name = $name ? $name : $decorator->getName();
        $this->decorators[$position][$name] = $decorator;

        return $this;
    }

    /**
     * @param boolean $position
     * @return mixed
     */
    public function getDecoratorsByPosition($position)
    {
        return isset($this->decorators[$position]) ? $this->decorators[$position] : [];
    }

    /**
     * @param $name
     * @param bool $position
     * @return bool
     */
    public function getDecorator($name, $position = false)
    {
        if ($position) {
            return $this->decorators[$position][$name];
        } else {
            foreach ($this->decorators as $position => $decorators) {
                if (isset($decorators[$name])) {
                    return $decorators[$name];
                }
            }
        }

        return false;
    }

    /**
     * @param $name
     * @param bool $position
     * @return $this
     */
    public function removeDecorator($name, $position = false)
    {
        if ($position) {
            unset($this->decorators[$position][$name]);
        } else {
            foreach ($this->decorators as $position => $decorators) {
                if (isset($decorators[$name])) {
                    unset($decorators[$name]);

                    return $this;
                }
            }
        }

        return $this;
    }
}