Comrade42\PhpBBParser\Command\ParseUsersCommand

Complexity

Total Complexity

8

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	6

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	4	1
B	execute()	0	60	7

<?php

namespace Comrade42\PhpBBParser\Command;

use Goutte\Client;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\DomCrawler\Crawler;

/**
 * Class ParseUsersCommand
 * @package Comrade42\PhpBBParser\Command
 */
class ParseUsersCommand extends ContainerAwareCommand
{
    const USERS_PER_PAGE = 50;

    protected function configure()
    {
        $this->setName('parse:users')->setDescription('Parse users');
    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        /** @var \Doctrine\ORM\EntityManager $entityManager */
        $entityManager = $this->container->get('doctrine');
        /** @var \Comrade42\PhpBBParser\Bridge\BridgeInterface $entityBridge */
        $entityBridge = $this->container->get('bridge');
        /** @var \Symfony\Component\Console\Helper\DialogHelper $dialogHelper */
        $dialogHelper = $this->getHelperSet()->get('dialog');

        $client = new Client();
        $baseUrl = rtrim($this->container->getParameter('forum_url'), '/');

        for ($offset = 0; ; $offset += static::USERS_PER_PAGE)
        {
            $url = $baseUrl . '/memberlist?mode=joined&start=' . $offset;
            $output->write(str_pad("<info>⇒ HTTP GET: {$url}</info> ", 90));

            $crawler = $client->request('GET', $url);

            $status = $client->getInternalResponse()->getStatus();
            $output->writeln("<info>[{$status}]</info>");

            if ($status != 200)
            {
                $offset -= static::USERS_PER_PAGE;
                sleep(1);
                continue;
            }

            $crawler = $crawler->filter('#memberlist tbody tr');
            $crawler->each(function (Crawler $node) use ($output, $entityManager, $entityBridge, $dialogHelper) {
                $columns = $node->children();
                $id = substr($columns->eq(1)->filter('a')->attr('href'), 2);
                $nickname = substr($columns->eq(1)->text(), 2);

                $entity = $entityBridge->getMemberEntity($entityManager, $id);
                $memberName = $entity->getNickname();

                $question = "<question>Member name doesn't match for #{$id} ({$memberName} → {$nickname}). Update entity? [Y/n]:</question> ";
                if (!empty($memberName) && $memberName != $nickname && !$dialogHelper->askConfirmation($output, $question)) return;

                $regDate = \DateTime::createFromFormat('Y-m-d H:i:s', $columns->eq(3)->text() . ' 00:00:00');
                $avatarUrl = $columns->eq(1)->filter('a img')->attr('src');
                $uuid = uniqid();

                $entity->setNickname($nickname)
                    ->setRealName($nickname)
                    ->setRegDate($regDate)

It seems like $regDate defined by \DateTime::createFromFormat('Y-m-d H:i:s', $columns->eq(3)->text() . ' 00:00:00') on line 64 can also be of type false; however, Comrade42\PhpBBParser\Entity\MemberInterface::setRegDate() does only seem to accept object<DateTime>, did you maybe forget to handle an error condition?

                    ->setPassword($uuid)
                    ->setEmailAddress($uuid . '@mailforspam.com')
                    ->setAvatarUrl($avatarUrl);

                $entityManager->persist($entity);
            });

            $entityManager->flush();

            if ($crawler->count() < static::USERS_PER_PAGE) break;
        }
    }
}