Developtech\AgilityBundle\Manager\FeedbackManager

Complexity

Total Complexity

7

Size/Duplication

Total Lines	80
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

90.48%

Importance

Changes	6
Bugs	0	Features	5

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getProjectFeedbacks()	0	3	1
A	getProjectFeedbacksByAuthor()	0	6	1
A	getFeedback()	0	6	2
A	createFeedback()	0	14	1
A	countFeedbacksPerStatus()	0	3	1

<?php

namespace Developtech\AgilityBundle\Manager;

use Doctrine\ORM\EntityManager;

use Developtech\AgilityBundle\Model\ProjectModel;
use Developtech\AgilityBundle\Model\FeedbackModel;

use Developtech\AgilityBundle\Entity\Feedback;

use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;

use Developtech\AgilityBundle\Utils\Slugger;

use Symfony\Component\Security\Core\User\UserInterface;

class FeedbackManager {
    /** @var EntityManager **/
    protected $em;
    /** @var Slugger **/
    protected $slugger;

    /**
     * @param EntityManager $em
     * @param Slugger $slugger
     */
    public function __construct(EntityManager $em, Slugger $slugger) {

You have injected the EntityManager via parameter $em. This is generally not recommended as it might get closed and become unusable. Instead, it is recommended to inject the ManagerRegistry and retrieve the EntityManager via getManager() each time you need it.

        $this->em = $em;
        $this->slugger = $slugger;
    }

    /**
     * @param ProjectModel $project
     * @return array
     */
    public function getProjectFeedbacks(ProjectModel $project) {
        return $this->em->getRepository(Feedback::class)->findByProject($project);
    }

    /**
    * @param ProjectModel $project
    * @param UserInterface $author
    * @param array $orderBy
    * @param integer $limit
    * @param integer $offset
    * @return array
    */
    public function getProjectFeedbacksByAuthor(ProjectModel $project, UserInterface $author, $orderBy = null, $limit = null, $offset = null) {
        return $this->em->getRepository(Feedback::class)->findBy([
            'project' => $project,
            'author' => $author
        ], $orderBy, $limit, $offset);
    }

    /**
     * @param integer $id
     * @return FeedbackModel
     */
    public function getFeedback($id) {
        if(($feedback = $this->em->getRepository(Feedback::class)->find($id)) === null) {

Expected 1 space after IF keyword; 0 found

            throw new NotFoundHttpException('Feedback not found');
        }
        return $feedback;
    }

    /**
     * @param ProjectModel $project
     * @param string $name
     * @param string $description
     * @param UserInterface $author
     * @return FeedbackModel
     */
    public function createFeedback(ProjectModel $project, $name, $description, UserInterface $author) {
        $feedback =
            (new Feedback())
            ->setName($name)
            ->setSlug($this->slugger->slugify($name))
            ->setDescription($description)
            ->setProject($project)
            ->setAuthor($author)
            ->setStatus(FeedbackModel::STATUS_OPEN)
        ;
        $this->em->persist($feedback);
        $this->em->flush();
        return $feedback;
    }

    /**
     * @param ProjectModel $project
     * @param integer $status
     * @return integer
     */
    public function countFeedbacksPerStatus(ProjectModel $project, $status) {
        return $this->em->getRepository(Feedback::class)->countPerStatus($project, $status);
    }
}