Dontdrinkandroot\Repository\TransactionalCrudRepository

Complexity

Total Complexity

18

Size/Duplication

Total Lines	197
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

71.23%

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	2
A	find()	0	8	1
A	findAll()	0	8	1
A	findBy()	0	8	1
A	findOneBy()	0	8	1
A	persist()	0	11	1
A	merge()	0	9	1
A	remove()	0	9	1
A	removeById()	0	11	2
A	count()	0	8	1
A	matching()	0	8	1
A	removeAll()	0	9	1
A	findPaginatedBy()	0	12	1
A	countAll()	0	8	1
A	getTransactionManager()	0	4	1
A	setTransactionManager()	0	4	1

<?php

namespace Dontdrinkandroot\Repository;

use Doctrine\Common\Collections\Criteria;
use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\Mapping;
use Doctrine\ORM\Tools\Pagination\Paginator;

/**
 * @author Philip Washington Sorst <philip@sorst.net>
 */
class TransactionalCrudRepository extends CrudRepository

There is one abstract method getClassName in this class; you could implement it, or declare this class as abstract.

{
    /** @var TransactionManager */
    private $transactionManager;

    public function __construct(
        EntityManagerInterface $em,
        Mapping\ClassMetadata $class,
        ?TransactionManager $transactionManager = null
    ) {
        parent::__construct($em, $class);

It seems like you code against a specific sub-type and not the parent class Dontdrinkandroot\Repository\CrudRepository as the method __construct() does only exist in the following sub-classes of Dontdrinkandroot\Repository\CrudRepository: Dontdrinkandroot\Repository\TransactionalCrudRepository, Dontdrinkandroot\Repository\TransactionalUuidCrudRepository. Maybe you want to instanceof check for one of these explicitly?

        if (null === $transactionManager) {
            $this->transactionManager = new TransactionManager($this->getEntityManager());

The method getEntityManager() does not seem to exist on object<Dontdrinkandroot\Repository\TransactionalCrudRepository>.

        } else {
            $this->transactionManager = $transactionManager;
        }
    }

    /**
     * {@inheritdoc}
     */
    public function find($id, $lockMode = null, $lockVersion = null)
    {
        return $this->transactionManager->transactional(
            function () use ($id, $lockMode, $lockVersion) {
                return parent::find($id, $lockMode = null, $lockVersion = null);

Consider using a different name than the imported variable $lockMode, or did you forget to import by reference?

Consider using a different name than the imported variable $lockVersion, or did you forget to import by reference?

The call to CrudRepository::find() has too many arguments starting with $lockMode = null.

            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function findAll()
    {
        return $this->transactionManager->transactional(
            function () {
                return parent::findAll();
            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function findBy(array $criteria, ?array $orderBy = null, $limit = null, $offset = null)
    {
        return $this->transactionManager->transactional(
            function () use ($criteria, $orderBy, $limit, $offset) {
                return parent::findBy($criteria, $orderBy, $limit, $offset);
            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function findOneBy(array $criteria, array $orderBy = null)
    {
        return $this->transactionManager->transactional(
            function () use ($criteria, $orderBy) {
                return parent::findOneBy($criteria, $orderBy);

The call to CrudRepository::findOneBy() has too many arguments starting with $orderBy.

            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function persist(object $entity, bool $flush = true): object
    {
        return $this->transactionManager->transactional(
            function () use ($entity) {
                $this->getEntityManager()->persist($entity);

The method getEntityManager() does not seem to exist on object<Dontdrinkandroot\Repository\TransactionalCrudRepository>.

                return $entity;
            },
            $flush
        );
    }

    /**
     * {@inheritdoc}
     */
    public function merge(object $entity, $flush = false): object
    {
        return $this->transactionManager->transactional(
            function () use ($entity) {
                $this->getEntityManager()->merge($entity);

The method getEntityManager() does not seem to exist on object<Dontdrinkandroot\Repository\TransactionalCrudRepository>.

            },
            $flush
        );
    }

    /**
     * {@inheritdoc}
     */
    public function remove(object $entity, bool $flush = false): void
    {
        $this->transactionManager->transactional(
            function () use ($entity): void {
                $this->getEntityManager()->remove($entity);

The method getEntityManager() does not seem to exist on object<Dontdrinkandroot\Repository\TransactionalCrudRepository>.

            },
            $flush
        );
    }

    /**
     * {@inheritdoc}
     */
    public function removeById($id, bool $flush = false): void
    {
        $this->transactionManager->transactional(
            function () use ($id, $flush): void {
                $entity = $this->find($id);
                if (null !== $entity) {
                    $this->remove($entity, $flush);
                }
            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function count(array $criteria)
    {
        return $this->transactionManager->transactional(
            function () use ($criteria) {
                return parent::count($criteria);

It seems like you code against a specific sub-type and not the parent class Dontdrinkandroot\Repository\CrudRepository as the method count() does only exist in the following sub-classes of Dontdrinkandroot\Repository\CrudRepository: Dontdrinkandroot\Repository\TransactionalCrudRepository, Dontdrinkandroot\Repository\TransactionalUuidCrudRepository. Maybe you want to instanceof check for one of these explicitly?

            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function matching(Criteria $criteria)
    {
        return $this->transactionManager->transactional(
            function () use ($criteria) {
                return parent::matching($criteria);

It seems like you code against a specific sub-type and not the parent class Dontdrinkandroot\Repository\CrudRepository as the method matching() does only exist in the following sub-classes of Dontdrinkandroot\Repository\CrudRepository: Dontdrinkandroot\Repository\TransactionalCrudRepository, Dontdrinkandroot\Repository\TransactionalUuidCrudRepository. Maybe you want to instanceof check for one of these explicitly?

            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function removeAll(bool $flush = false, bool $iterate = true): void
    {
        $this->transactionManager->transactional(
            function () use ($iterate): void {
                parent::removeAll(false, $iterate);
            },
            $flush
        );
    }

    /**
     * {@inheritdoc}
     */
    public function findPaginatedBy(
        int $page = 1,
        int $perPage = 10,
        array $criteria = [],
        array $orderBy = null
    ): Paginator {
        return $this->transactionManager->transactional(
            function () use ($page, $perPage, $criteria, $orderBy): Paginator {
                return parent::findPaginatedBy($page, $perPage, $criteria, $orderBy);
            }
        );
    }

    /**
     * {@inheritdoc}
     */
    public function countAll(): int
    {
        return $this->transactionManager->transactional(
            function (): int {
                return parent::countAll();
            }
        );
    }

    public function getTransactionManager(): TransactionManager
    {
        return $this->transactionManager;
    }

    public function setTransactionManager(TransactionManager $transactionManager): void
    {
        $this->transactionManager = $transactionManager;
    }
}