PiedWeb\SeoPocketCrawler\Command\ShowExternalLinksCommand::execute()

Complexity

Conditions	8
Paths	4

Size

Total Lines	28
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $table = new Table($output);

        $table->setHeaders($input->getOption('host') ? ['Host'] : ['url', 'from']);

        $links = ExtractExternalLinks::scan($input->getArgument('id'));

It seems like $input->getArgument('id') can also be of type null and string[]; however, parameter $id of PiedWeb\SeoPocketCrawler\ExtractExternalLinks::scan() does only seem to accept string, maybe add an additional type check?

        arsort($links);
        $ever = [];
        foreach ($links as $link => $from) {
            if ($input->getOption('host')) {
                $host = parse_url($link, PHP_URL_HOST);
                if ($host && !isset($ever[$host])) {
                    $ever[$host] = 1;
                    $table->addRow([$host]);
                }
            } else {
                $first = true;
                foreach ($from as $url) {
                    $table->addRow([true === $first ? $link : '', $url]);
                    $first = false;
                }
            }
        }

        $table->render();

        return 0;
    }