MageScan\Command\Scan\SitemapCommand

Complexity

Total Complexity

5

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
A	execute()	0	22	3
A	getSitemapUrl()	0	11	1

<?php
/**
 * Mage Scan
 *
 * PHP version 5
 *
 * @category  MageScan
 * @package   MageScan
 * @author    Steve Robbins <steve@steverobbins.com>
 * @copyright 2015 Steve Robbins
 * @license   http://creativecommons.org/licenses/by/4.0/ CC BY 4.0
 * @link      https://github.com/steverobbins/magescan
 */

namespace MageScan\Command\Scan;

use MageScan\Check\Sitemap;
use MageScan\Request;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Scan sitemap command
 *
 * @category  MageScan
 * @package   MageScan
 * @author    Steve Robbins <steve@steverobbins.com>
 * @copyright 2015 Steve Robbins
 * @license   http://creativecommons.org/licenses/by/4.0/ CC BY 4.0
 * @link      https://github.com/steverobbins/magescan
 */
class SitemapCommand extends AbstractCommand
{
    /**
     * Configure command
     *
     * @return void
     */
    protected function configure()
    {
        $this
            ->setName('scan:sitemap')
            ->setDescription('Check sitemap');
        parent::configure();
    }

    /**
     * Execute command
     *
     * @param InputInterface  $input
     * @param OutputInterface $output
     *
     * @return void
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $result = [];
        $url = $this->getSitemapUrl();
        if ($url === false) {
            $result[] = '<error>Sitemap is not declared in robots.txt</error>';
            $url = $this->request->getUrl() . 'sitemap.xml';
        } else {
            $result[] = '<info>Sitemap is declared in robots.txt</info>';
        }
        $request = new Request(
            $url,
            $this->input->getOption('insecure')
        );
        $response = $request->get();
        if ($response->getStatusCode() == 200) {
            $result[] = '<info>Sitemap is accessible:</info> ' . $url;
        } else {
            $result[] = '<error>Sitemap is not accessible:</error> ' . $url;
        }
        $this->out('Sitemap', $result);
    }

    /**
     * Parse the robots.txt text file to find the sitemap
     *
     * @return string|boolean
     */
    protected function getSitemapUrl()
    {
        $request = new Request(
            $this->request->getUrl(),

It seems like $this->request->getUrl() targeting MageScan\Request::getUrl() can also be of type boolean; however, MageScan\Request::__construct() does only seem to accept false|string, maybe add an additional type check?

            $this->input->getOption('insecure')
        );
        $response = $request->get('robots.txt');
        $sitemap = new Sitemap;
        $sitemap->setRequest($this->request);
        return $sitemap->getSitemapFromRobotsTxt($response);
    }
}