Infrastructure\Persistence\DoctrineORM\Repository\TaskRepository

Complexity

Total Complexity

13

Size/Duplication

Total Lines	110
Duplicated Lines	29.09 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	findAll()	0	4	1
A	find()	0	11	2
A	findAllByStatus()	0	8	1
A	findByName()	0	15	2
A	save()	16	16	3
A	remove()	16	16	3
A	removeByStatus()	0	9	1

<?php

namespace Todo\Infrastructure\Persistence\DoctrineORM\Repository;

use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\OptimisticLockException;
use Doctrine\ORM\ORMInvalidArgumentException;
use Todo\Domain\Exception\TaskNotFoundException;
use Todo\Domain\Repository\TaskRepositoryInterface;
use Todo\Domain\Task;

/**
 * Class TaskRepository
 *
 * @category None
 * @package  Infrastructure\Persistence\DoctrineORM\Repository
 * @author   Martin Pham <i@martinpham.com>
 * @license  None http://
 * @link     None
 */
class TaskRepository extends EntityRepository implements TaskRepositoryInterface
{
    /**
     * @inheritDoc
     */
    public function findAll(): array
    {
        return parent::findAll();
    }/** @noinspection PhpSignatureMismatchDuringInheritanceInspection */

    /**
     * @inheritDoc
     */
    public function find($id): Task
    {
        /** @var Task $task */
        $task = parent::find($id);

        if ($task === null) {
            throw new TaskNotFoundException("Cannot find task with id $id");
        }

        return $task;
    }

    /**
     * @inheritDoc
     */
    public function findAllByStatus($status): array
    {
        return $this->findBy(
            [
            'status' => $status
            ]
        );
    }

    /**
     * @inheritDoc
     */
    public function findByName(string $name): Task
    {
        /** @var Task $task */
        $task = $this->findOneBy(
            [
                'name' => $name
            ]
        );

        if ($task === null) {
            throw new TaskNotFoundException("Cannot find task with name $name");
        }

        return $task;
    }

    /**
     * @inheritDoc
     */
    public function save(Task $task): bool

This method seems to be duplicated in your project.

    {
        try {
            $this->getEntityManager()->persist($task);
        } catch (ORMInvalidArgumentException $e) {
            return false;
        }

        try {
            $this->getEntityManager()->flush();
        } catch (OptimisticLockException $e) {
            return false;
        }

        return true;
    }

    /**
     * @inheritDoc
     */
    public function remove(Task $task): bool

This method seems to be duplicated in your project.

    {
        try {
            $this->getEntityManager()->remove($task);
        } catch (ORMInvalidArgumentException $e) {
            return false;
        }

        try {
            $this->getEntityManager()->flush();
        } catch (OptimisticLockException $e) {
            return false;
        }

        return true;
    }

    /**
     * @inheritDoc
     */
    public function removeByStatus($status): bool
    {
        $query = $this->getEntityManager()
            ->createQuery('DELETE FROM Domain\Task t WHERE t.status = :status');
        $query->setParameter('status', $status);
        $query->execute();

        return true;
    }

}