VideoGamesRecords\CoreBundle\Ranking\Provider\Player\PlayerCountryRankingProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getRankingPoints()	0	24	3
A	getRankingMedals()	0	3	1

<?php

namespace VideoGamesRecords\CoreBundle\Ranking\Provider\Player;

use VideoGamesRecords\CoreBundle\Ranking\Provider\AbstractRankingProvider;

class PlayerCountryRankingProvider extends AbstractRankingProvider
{
    /**
     * @param int|null $id
     * @param array    $options
     * @return array
     */
    public function getRankingPoints(int $id = null, array $options = []): array
    {
        $country = $this->em->getRepository('VideoGamesRecords\CoreBundle\Entity\Country')->find($id);
        if (null === $country) {
            return [];
        }

        $maxRank = $options['maxRank'] ?? null;

        $query = $this->em->createQueryBuilder('p')

The call to Doctrine\ORM\EntityManagerInterface::createQueryBuilder() has too many arguments starting with 'p'.

            ->select('p')
            ->from('VideoGamesRecords\CoreBundle\Entity\Player', 'p')
            ->where('(p.country = :country)')
            ->setParameter('country', $country)
            ->orderBy('p.rankCountry');

        if ($maxRank !== null) {
            $query->andWhere('p.rankCountry <= :maxRank')
                ->setParameter('maxRank', $maxRank);
        } else {
            $query->setMaxResults($maxRank);
        }

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

The expression return $query->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.

    }

    /**
     * @param int|null $id
     * @param array $options
     * @return array
     */
    public function getRankingMedals(int $id = null, array $options = []): array
    {
        return array();
    }
}