VideoGamesRecords\CoreBundle\Handler\Badge\PlayerMasterBadgeHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	process()	0	10	2
A	__construct()	0	4	1

<?php

namespace VideoGamesRecords\CoreBundle\Handler\Badge;

use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\Exception\ORMException;
use VideoGamesRecords\CoreBundle\Entity\Game;
use VideoGamesRecords\CoreBundle\Service\Ranking\Read\PlayerGameRankingQuery;

class PlayerMasterBadgeHandler
{
    private EntityManagerInterface $em;
    private PlayerGameRankingQuery $playerGameRankingQuery;

    public function __construct(EntityManagerInterface $em, PlayerGameRankingQuery $playerGameRankingQuery)
    {
        $this->em = $em;
        $this->playerGameRankingQuery = $playerGameRankingQuery;
    }

    /**
     * @throws ORMException
     */
    public function process(Game $game): void
    {
        //----- get ranking with maxRank = 1
        $ranking = $this->playerGameRankingQuery->getRankingPoints($game->getId(), ['maxRank' => 1]);
        $players = array();
        foreach ($ranking as $playerGame) {
            $players[$playerGame->getPlayer()->getId()] = 0;
        }

        $this->em->getRepository('VideoGamesRecords\CoreBundle\Entity\PlayerBadge')->updateBadge($players, $game->getBadge());

It seems like $game->getBadge() can also be of type null; however, parameter $badge of VideoGamesRecords\CoreBundle\Repository\PlayerBadgeRepository::updateBadge() does only seem to accept VideoGamesRecords\CoreBundle\Entity\Badge, maybe add an additional type check?

    }
}