Mykees\TagBundle\Traits\TaggableTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes	4
Bugs	1	Features	2

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getModel()	0	4	1
A	getModelId()	0	4	1
A	setTags()	0	4	2
A	getTags()	0	6	2
A	getRemove()	0	4	1
A	setRemove()	0	5	1

<?php
/**
 * Created by PhpStorm.
 * User: Rafidion Michael
 * Date: 10/12/2014
 * Time: 14:08
 */

namespace Mykees\TagBundle\Traits;


use Doctrine\Common\Collections\ArrayCollection;
use Mykees\TagBundle\Util\Reflection;

trait TaggableTrait {

    public $tags;
    public $remove = false;

    public function getModel()
    {
        return Reflection::getClassShortName($this);
    }

    public function getModelId()
    {
        return $this->getId();

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

    }


    public function setTags($tags)
    {
        $this->tags = !empty($tags) ? new ArrayCollection($tags) : null;
    }

    public function getTags()
    {
        $this->tags = $this->tags ?: new ArrayCollection();
        
        return $this->tags;
    }

    public function getRemove()
    {
        return $this->remove;
    }

    public function setRemove($remove)
    {
        $this->remove = $remove;
        return $this->remove;
    }
}