VideoGamesRecords\CoreBundle\Handler\Video\YoutubeDataHandler::process()

Complexity

Conditions	4
Paths	17

Size

Total Lines	31
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function process(Video $video): void
    {
        if ($video->getType()->getValue() !== VideoType::TYPE_YOUTUBE) {
            throw new \InvalidArgumentException();
        }

        try {
            $response = $this->youtubeProvider->getVideo($video->getVideoId());
            if (count($response->getItems()) > 0) {
                $youtubeVideo = $response->getItems()[0];

                $snippet = $youtubeVideo->getSnippet();
                $video->setTitle($snippet->getTitle());
                echo $video->getId() . "\n";
                //var_dump($snippet->getThumbnails());
                $video->setThumbnail(
                    $snippet->getThumbnails()
                        ->getHigh()
                        ->getUrl()
                );

                $video->setDescription($snippet->getDescription());

                $statistics = $youtubeVideo->getStatistics();
                $video->setLikeCount($statistics->getLikeCount() ?? 0);

It seems like $statistics->getLikeCount() ?? 0 can also be of type string; however, parameter $likeCount of VideoGamesRecords\CoreBundle\Entity\Video::setLikeCount() does only seem to accept integer, maybe add an additional type check?

                $video->setViewCount($statistics->getViewCount() ?? 0);

It seems like $statistics->getViewCount() ?? 0 can also be of type string; however, parameter $viewCount of VideoGamesRecords\CoreBundle\Entity\Video::setViewCount() does only seem to accept integer, maybe add an additional type check?

            } else {
                $video->setIsActive(false);
            }
            $this->em->flush();
        } catch (\Exception $e) {

Consider adding a comment why this CATCH block is empty.