App\Movies\Command\LoadWatchCards

Complexity

Total Complexity

6

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	9

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	configure()	0	7	1
A	execute()	0	33	4

<?php

namespace App\Movies\Command;

use App\Movies\Entity\Movie;
use App\Movies\Entity\MovieCard;
use App\Movies\Parser\Megogo;
use App\Movies\Repository\MovieCardRepository;
use App\Movies\Repository\MovieRepository;
use App\Users\Repository\UserRepository;
use Doctrine\ORM\EntityManagerInterface;
use Psr\SimpleCache\CacheInterface;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class LoadWatchCards extends Command
{
    /** @var $repository MovieRepository */
    private $repository;

    private $userRepository;

    /** @var $parser Megogo */
    private $parser;

    private $em;
    private $cache;

    private const CACHE_TIME = 2592000; // 30 days

    public function __construct(EntityManagerInterface $em, UserRepository $userRepository, MovieCardRepository $cardRepository, MovieRepository $repository, Megogo $parser, CacheInterface $cache, ?string $name = null)
    {
        parent::__construct($name);

        $this->repository = $repository;
        $this->cardRepository = $cardRepository;

The property cardRepository does not seem to exist. Did you mean repository?

        $this->userRepository = $userRepository;
        $this->parser = $parser;
        $this->cache = $cache;
        $this->em = $em;
    }

    protected function configure()
    {
        $this
            ->setName('app:load-watch-cards')
            ->setDescription('Find movies on megogo')
            ->setHelp('php bin/console app:load-watch-cards');
    }

    protected function execute(InputInterface $i, OutputInterface $o)
    {
        $title = 'Megogo';
        $locale = 'uk';
        $movies = $this->repository->findAllWithoutCardsQuery($title, $locale)->getResult();
        $user = $this->userRepository->findOneBy([
            'username' => 'support'
        ]);

        /** @var $movie Movie */
        foreach ($movies as $movie) {
            $cacheKey = 'megogo_notfound_' . $movie->getId();
            if ($this->cache->has($cacheKey)) {
                continue;
            }

            try {
                $url = $this->parser->getUrlByTitle($movie->getOriginalTitle());
            } catch (\ErrorException $e) {

The class ErrorException does not exist. Did you forget a USE statement, or did you not list all dependencies?

                $this->cache->set($cacheKey, true, self::CACHE_TIME);
                $o->writeln(sprintf(
                    '[LoadWatchCards] [%s]: %s', $movie->getOriginalTitle(), $e->getMessage()
                ));
                continue;
            }

            $description = sprintf('Дивитись онлайн в хорошій якості у найбільшому онлайн кінотеатрі у східній Європі');
            $card = new MovieCard($movie, $user, $locale, $title, $description, MovieCard::TYPE_WATCH, $url);

It seems like $user defined by $this->userRepository->findOneBy(array('username' => 'support')) on line 57 can be null; however, App\Movies\Entity\MovieCard::__construct() does not accept null, maybe add an additional type check?

            $this->em->persist($card);
            $this->em->flush();
        }
    }
}