From e67b03d88a517c04f49171c23526f7d7eb6d0ec1 Mon Sep 17 00:00:00 2001 From: Mathias Arlaud Date: Wed, 27 Dec 2023 11:39:16 +0100 Subject: [PATCH] fix: rely on doctrine paginator for counting --- .../Infrastructure/Doctrine/DoctrineRepository.php | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/Shared/Infrastructure/Doctrine/DoctrineRepository.php b/src/Shared/Infrastructure/Doctrine/DoctrineRepository.php index eda69f0..dd84c0e 100644 --- a/src/Shared/Infrastructure/Doctrine/DoctrineRepository.php +++ b/src/Shared/Infrastructure/Doctrine/DoctrineRepository.php @@ -46,14 +46,9 @@ public function getIterator(): \Iterator public function count(): int { - if (null !== $paginator = $this->paginator()) { - return count($paginator); - } + $paginator = $this->paginator() ?? new Paginator(clone $this->queryBuilder); - return (int) (clone $this->queryBuilder) - ->select('count(1)') - ->getQuery() - ->getSingleScalarResult(); + return count($paginator); } public function paginator(): ?PaginatorInterface