Skip to content

Commit

Permalink
refactor: Added missing use statements + clean up for non related exc…
Browse files Browse the repository at this point in the history
…eptions.
  • Loading branch information
tarlepp committed Jul 5, 2019
1 parent 274c4b2 commit 07ee575
Showing 1 changed file with 20 additions and 17 deletions.
37 changes: 20 additions & 17 deletions src/Repository/Traits/RepositoryMethodsTrait.php
Expand Up @@ -13,8 +13,14 @@
use App\Rest\RepositoryHelper; use App\Rest\RepositoryHelper;
use Doctrine\ORM\EntityManager; use Doctrine\ORM\EntityManager;
use Doctrine\ORM\EntityRepository; use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\NonUniqueResultException;
use Doctrine\ORM\OptimisticLockException;
use Doctrine\ORM\ORMException;
use Doctrine\ORM\ORMInvalidArgumentException;
use Doctrine\ORM\QueryBuilder; use Doctrine\ORM\QueryBuilder;
use Doctrine\ORM\Tools\Pagination\Paginator; use Doctrine\ORM\Tools\Pagination\Paginator;
use Doctrine\ORM\TransactionRequiredException;
use InvalidArgumentException;
use function array_map; use function array_map;
use function array_values; use function array_values;


Expand All @@ -41,10 +47,10 @@ trait RepositoryMethodsTrait
* *
* @return EntityInterface|mixed|null * @return EntityInterface|mixed|null
* *
* @throws \Doctrine\ORM\TransactionRequiredException * @throws TransactionRequiredException
* @throws \Doctrine\ORM\OptimisticLockException * @throws OptimisticLockException
* @throws \Doctrine\ORM\ORMInvalidArgumentException * @throws ORMInvalidArgumentException
* @throws \Doctrine\ORM\ORMException * @throws ORMException
*/ */
public function find(string $id, ?int $lockMode = null, ?int $lockVersion = null) public function find(string $id, ?int $lockMode = null, ?int $lockVersion = null)
{ {
Expand All @@ -57,15 +63,12 @@ public function find(string $id, ?int $lockMode = null, ?int $lockVersion = null
* Advanced version of find method, with this you can process query as you like, eg. add joins and callbacks to * Advanced version of find method, with this you can process query as you like, eg. add joins and callbacks to
* modify / optimize current query. * modify / optimize current query.
* *
* @psalm-suppress LessSpecificImplementedReturnType
*
* @param string $id * @param string $id
* @param string|int $hydrationMode * @param string|int $hydrationMode
* *
* @return array<int|string, mixed>|EntityInterface * @return array<int|string, mixed>|EntityInterface
* *
* @throws \Doctrine\ORM\NoResultException * @throws NonUniqueResultException
* @throws \Doctrine\ORM\NonUniqueResultException
*/ */
public function findAdvanced(string $id, $hydrationMode = null) public function findAdvanced(string $id, $hydrationMode = null)
{ {
Expand Down Expand Up @@ -134,7 +137,7 @@ public function findBy(array $criteria, ?array $orderBy = null, ?int $limit = nu
* *
* @return array<EntityInterface>|EntityInterface[] * @return array<EntityInterface>|EntityInterface[]
* *
* @throws \InvalidArgumentException * @throws InvalidArgumentException
*/ */
public function findByAdvanced( public function findByAdvanced(
array $criteria, array $criteria,
Expand Down Expand Up @@ -183,7 +186,7 @@ public function findAll(): array
* *
* @return string[] * @return string[]
* *
* @throws \InvalidArgumentException * @throws InvalidArgumentException
*/ */
public function findIds(?array $criteria = null, ?array $search = null): array public function findIds(?array $criteria = null, ?array $search = null): array
{ {
Expand Down Expand Up @@ -217,8 +220,8 @@ public function findIds(?array $criteria = null, ?array $search = null): array
* *
* @return int * @return int
* *
* @throws \InvalidArgumentException * @throws InvalidArgumentException
* @throws \Doctrine\ORM\NonUniqueResultException * @throws NonUniqueResultException
*/ */
public function countAdvanced(?array $criteria = null, ?array $search = null): int public function countAdvanced(?array $criteria = null, ?array $search = null): int
{ {
Expand Down Expand Up @@ -268,8 +271,8 @@ public function reset(): int
* *
* @return BaseRepositoryInterface * @return BaseRepositoryInterface
* *
* @throws \Doctrine\ORM\ORMException * @throws ORMException
* @throws \Doctrine\ORM\OptimisticLockException * @throws OptimisticLockException
*/ */
public function save(EntityInterface $entity, ?bool $flush = null): BaseRepositoryInterface public function save(EntityInterface $entity, ?bool $flush = null): BaseRepositoryInterface
{ {
Expand All @@ -294,8 +297,8 @@ public function save(EntityInterface $entity, ?bool $flush = null): BaseReposito
* *
* @return BaseRepositoryInterface * @return BaseRepositoryInterface
* *
* @throws \Doctrine\ORM\ORMException * @throws ORMException
* @throws \Doctrine\ORM\OptimisticLockException * @throws OptimisticLockException
*/ */
public function remove(EntityInterface $entity, ?bool $flush = null): BaseRepositoryInterface public function remove(EntityInterface $entity, ?bool $flush = null): BaseRepositoryInterface
{ {
Expand Down Expand Up @@ -323,7 +326,7 @@ public function remove(EntityInterface $entity, ?bool $flush = null): BaseReposi
* *
* @return QueryBuilder * @return QueryBuilder
* *
* @throws \InvalidArgumentException * @throws InvalidArgumentException
*/ */
private function getQueryBuilder( private function getQueryBuilder(
?array $criteria = null, ?array $criteria = null,
Expand Down

0 comments on commit 07ee575

Please sign in to comment.