Matks\Bundle\CustomerSupportBundle\Entity\Category

Complexity

Total Complexity

9

Size/Duplication

Total Lines	104
Duplicated Lines	0 %

Test Coverage

Coverage

86.96%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getTickets()	0	3	1
A	activate()	0	7	2
A	isActive()	0	3	1
A	getTitle()	0	3	1
A	deactivate()	0	11	3

<?php

namespace Matks\Bundle\CustomerSupportBundle\Entity;

use Doctrine\ORM\Mapping as ORM;
use Knp\DoctrineBehaviors\Model as ORMBehaviors;
use Symfony\Component\Validator\Constraints as Assert;

use Matks\Bundle\CustomerSupportBundle\Model\TicketInterface;
use Matks\Bundle\CustomerSupportBundle\Model\CategoryInterface;

use Doctrine\Common\Collections\ArrayCollection;
use LogicException;

/**
 * Category entity
 *
 * @ORM\Entity(repositoryClass="Matks\Bundle\CustomerSupportBundle\Repository\CategoryRepository")
 * @ORM\Table(name="categories")
 *
 * @author Mathieu Ferment <mathieu.ferment@gmail.com>
 */
class Category implements CategoryInterface
{
    use ORMBehaviors\Timestampable\Timestampable;

    /**
     * @var int
     *
     * @ORM\Id
     * @ORM\Column(type="integer")
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;

The private property $id is not used, and could be removed.

    /**
     * @var string
     *
     * @ORM\Column(type="string", length=255)
     * @Assert\NotBlank()
     */
    private $title;

    /**
     * @var TicketInterface[]
     *
     * There are no addTicket or removeTicket functions as this is done by
     * the owning side, on Ticket entity
     *
     * @ORM\OneToMany(
     *     targetEntity="Matks\Bundle\CustomerSupportBundle\Model\TicketInterface",
     *     mappedBy="category",
     *     cascade={"persist", "remove", "merge"}
     * )
     * @Assert\Valid()
     */
    private $tickets;

    /**
     * @var boolean
     *
     * @ORM\Column(type="boolean")
     * @Assert\NotBlank()
     */
    private $enabled;

    /**
     * Constructor
     *
     * @param string $title
     */
    public function __construct($title)
    {
        $this->title   = $title;
        $this->tickets = new ArrayCollection();

It seems like new Doctrine\Common\Collections\ArrayCollection() of type Doctrine\Common\Collections\ArrayCollection is incompatible with the declared type Matks\Bundle\CustomerSupportBundle\Model\TicketInterface[] of property $tickets.

        $this->enabled = true;
    }

    /**
     * Get tickets associated
     *
     * @return TicketInterface[]
     */
    public function getTickets()
    {
        return $this->tickets;
    }

    /**
     * Get title
     *
     * @return string
     */
    public function getTitle()
    {
        return $this->title;
    }

    public function deactivate()
    {
        if (!$this->isActive()) {
            throw new LogicException("Cannot deactivate category not active");
        }

        if (!$this->getTickets()->isEmpty()) {
            throw new LogicException("Cannot deactivate category with tickets associated");
        }

        $this->enabled = false;
    }

    public function activate()
    {
        if ($this->isActive()) {
            throw new LogicException("Cannot activate category already active");
        }

        $this->enabled = true;
    }

    /**
     * @return bool
     */
    public function isActive()
    {
        return true === $this->enabled;
    }
}