VideoGamesRecords\CoreBundle\EventSubscriber\InsertLostPositionSubscriber

Complexity

Total Complexity

12

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
B	insertLostPosition()	0	18	10
A	getSubscribedEvents()	0	4	1

<?php

declare(strict_types=1);

namespace VideoGamesRecords\CoreBundle\EventSubscriber;

use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\Exception\ORMException;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use VideoGamesRecords\CoreBundle\Entity\Chart;
use VideoGamesRecords\CoreBundle\Entity\LostPosition;
use VideoGamesRecords\CoreBundle\Entity\Player;
use VideoGamesRecords\CoreBundle\Event\PlayerChartEvent;
use VideoGamesRecords\CoreBundle\VideoGamesRecordsCoreEvents;

final class InsertLostPositionSubscriber implements EventSubscriberInterface
{
    private EntityManagerInterface $em;

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

    public static function getSubscribedEvents(): array
    {
        return [
            VideoGamesRecordsCoreEvents::PLAYER_CHART_MAJ_COMPLETED => 'insertLostPosition',
        ];
    }

    /**
     * @param PlayerChartEvent $event
     * @throws ORMException
     */
    public function insertLostPosition(PlayerChartEvent $event): void
    {
        $playerChart = $event->getPlayerChart();
        $oldRank = $event->getOldRank();
        $oldNbEqual = $event->getOldNbEqual();
        $newRank = $playerChart->getRank();
        $newNbEqual = $playerChart->getNbEqual();

        if (
            (($oldRank >= 1) && ($oldRank <= 3) && ($newRank > $oldRank)) ||
            (($oldRank === 1) && ($oldNbEqual === 1) && ($newRank === 1) && ($newNbEqual > 1))
        ) {
            $lostPosition = new LostPosition();
            $lostPosition->setNewRank($newRank);

It seems like $newRank can also be of type null; however, parameter $newRank of VideoGamesRecords\CoreBundle\Entity\LostPosition::setNewRank() does only seem to accept integer, maybe add an additional type check?

            $lostPosition->setOldRank(($oldNbEqual == 1 && $oldRank == 1) ? 0 : $oldRank); //----- zero for losing platinum medal

It seems like $oldNbEqual == 1 && $oldRank == 1 ? 0 : $oldRank can also be of type null; however, parameter $oldRank of VideoGamesRecords\CoreBundle\Entity\LostPosition::setOldRank() does only seem to accept integer, maybe add an additional type check?

            $lostPosition->setPlayer($this->em->getReference(Player::class, $playerChart->getPlayer()->getId()));

It seems like $this->em->getReference(VideoGamesRecords\CoreBundle\Entity\Player::class, $playerChart->getPlayer()->getId()) can also be of type null; however, parameter $player of VideoGamesRecords\CoreBundle\Entity\LostPosition::setPlayer() does only seem to accept VideoGamesRecords\CoreBundle\Entity\Player, maybe add an additional type check?

            $lostPosition->setChart($this->em->getReference(Chart::class, $playerChart->getChart()->getId()));

It seems like $this->em->getReference(VideoGamesRecords\CoreBundle\Entity\Chart::class, $playerChart->getChart()->getId()) can also be of type null; however, parameter $chart of VideoGamesRecords\CoreBundle\Entity\LostPosition::setChart() does only seem to accept VideoGamesRecords\CoreBundle\Entity\Chart, maybe add an additional type check?

            $this->em->persist($lostPosition);
        }
    }
}