Application\Repository\DomainRepository

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Test Coverage

Coverage

91.67%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getByCards()	0	17	2

<?php

declare(strict_types=1);

namespace Application\Repository;

use Application\Model\Card;
use Application\Model\Domain;
use Doctrine\ORM\QueryBuilder;

/**
 * @extends AbstractHasParentRepository<Domain>
 */
class DomainRepository extends AbstractHasParentRepository
{
    public function getByCards(QueryBuilder $cardQb): array
    {
        $cardIds = array_map(fn ($card) => $card->getId(), $cardQb->getQuery()->getResult());

It seems like $cardQb->getQuery()->getResult() can also be of type integer; however, parameter $array of array_map() does only seem to accept array, maybe add an additional type check?

        if (count($cardIds) === 0) {
            return [];
        }

        $qb = $this
            ->createQueryBuilder('d')
            ->innerJoin(Card::class, 'c')
            ->innerJoin('c.domains', 'd2', 'WITH', 'd2.id = d.id')
            ->where('c.id in (' . implode(',', $cardIds) . ')')
            ->groupBy('d.id')
            ->orderBy('COUNT(c.id)', 'DESC');

        return $qb->getQuery()->getResult();

The expression return $qb->getQuery()->getResult() could return the type integer which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

    }
}