MageDownload\Command\VersionsCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
A	execute()	0	9	1
A	render()	0	19	2

<?php
/**
 * Magedownload CLI
 *
 * PHP version 5
 *
 * @category  MageDownload
 * @package   MageDownload
 * @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/magedownload-cli
 */

namespace MageDownload\Command;

use MageDownload\Info;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Versions command
 *
 * @category  MageDownload
 * @package   MageDownload
 * @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/magedownload-cli
 */
class VersionsCommand extends AbstractCommand
{
    const NAME = 'versions';

    const API_ACTION = 'versions';

    /**
     * Configure command
     *
     * @return void
     */
    protected function configure()
    {
        $this
            ->setName(self::NAME)
            ->setDescription('List files available for download');
        parent::configure();
    }

    /**
     * Execute command
     *
     * @param InputInterface  $input
     * @param OutputInterface $output
     *
     * @return void
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $info = new Info;
        return $this->render($info->sendCommand(
            self::API_ACTION,
            $this->getAccountId(),

It seems like $this->getAccountId() targeting MageDownload\Command\AbstractCommand::getAccountId() can also be of type boolean; however, MageDownload\Info::sendCommand() does only seem to accept string, maybe add an additional type check?

            $this->getAccessToken()

It seems like $this->getAccessToken() targeting MageDownload\Command\AbstractCommand::getAccessToken() can also be of type boolean; however, MageDownload\Info::sendCommand() does only seem to accept string, maybe add an additional type check?

        ));
    }

    /**
     * Render the versions action
     *
     * @param string $result
     *
     * @return void
     */
    protected function render($result)
    {
        $editions = preg_split('/\n{2}/', $result);
        foreach ($editions as $info) {
            $bits = preg_split('/\-{5,}/', $info);
            $versions = explode("\n", trim($bits[1]));
            usort($versions, 'version_compare');
            array_walk($versions, function (&$value) {
                $value = array($value);
            });
            $this->out(array(array(
                'type' => 'table',
                'data' => array(
                    array(trim($bits[0])),
                    $versions
                )
            )));
        }
    }
}