Nip\Form\Elements\Traits\HasUniqueIdTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

80.95%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getJSID()	0	6	1
A	getUniqueId()	0	8	2
A	setUniqueID()	0	4	1
A	initUniqueId()	0	4	1
A	generateUniqueId()	0	12	2

<?php

namespace Nip\Form\Elements\Traits;

/**
 * Trait HasUniqueIdTrait
 * @package Nip\Form\Elements\Traits
 */
trait HasUniqueIdTrait
{
    /**
     * @var null|string
     */
    protected $_uniqueID = null;

    /**
     * @return string
     */
    public function getJSID()
    {
        $name = $this->getUniqueId();

        return str_replace(['][', '[', ']'], ['-', '-', ''], $name);
    }

    /**
     * @return null|string
     */
    public function getUniqueId()
    {
        if (!$this->_uniqueID) {
            $this->initUniqueId();
        }

        return $this->_uniqueID;
    }

    /**
     * @param null|string $uniqueID
     */
    public function setUniqueID($uniqueID)
    {
        $this->_uniqueID = $uniqueID;
    }

    protected function initUniqueId()
    {
        $this->setUniqueID($this->generateUniqueId());
    }

    /**
     * @return null|string
     */
    protected function generateUniqueId()
    {
        $name = $this->getName();

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

        $registeredNames = (array)$this->getForm()->getCache('elements_names');
        if (in_array($name, $registeredNames)) {
            $name = uniqid($name);
        }
        $registeredNames[] = $name;
        $this->getForm()->setCache('elements_names', $registeredNames);

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

        return $name;
    }
}