Buttress\Concrete\Console\Command\Package\InstallController

Complexity

Total Complexity

16

Size/Duplication

Total Lines	114
Duplicated Lines	33.33 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	listPackages()	0	5	1
A	install()	19	19	3
A	uninstall()	19	19	3
A	getDefinition()	0	10	3
A	getAvailablePackages()	0	10	4
A	confirmHandle()	0	10	1

<?php

namespace Buttress\Concrete\Console\Command\Package;

use Buttress\Concrete\CommandBus\Command\Package\Install;
use Buttress\Concrete\CommandBus\Command\Package\ListPackages;
use Buttress\Concrete\CommandBus\Command\Package\Uninstall;
use Buttress\Concrete\Console\Command\PackageCommand;
use Buttress\Concrete\Locator\Site;
use League\CLImate\CLImate;
use League\Tactician\CommandBus;

class InstallController
{

    /** @var \Buttress\Concrete\Console\Command\PackageCommand */
    private $command;

    /** @var \League\Tactician\CommandBus */
    private $bus;

    /** @var \League\CLImate\CLImate */
    private $cli;

    public function __construct(PackageCommand $command, CommandBus $bus, CLImate $cli)
    {
        $this->command = $command;
        $this->bus = $bus;
        $this->cli = $cli;
    }

    /**
     * Handle package:list
     *
     * @param \Buttress\Concrete\Locator\Site $site
     */
    public function listPackages(Site $site)

The parameter $site is not used and could be removed.

    {
        $command = new ListPackages();
        $this->bus->handle($command);
    }

    /**
     * Handle package:install
     *
     * @param \Buttress\Concrete\Locator\Site $site
     */
    public function install(Site $site)

This method seems to be duplicated in your project.

    {
        $definition = $this->getDefinition($site, 'package:install');
        $definition->parse();
        $handle = $definition->get('handle');

        if (!$handle) {
            $packages = iterator_to_array($this->getAvailablePackages($site));
            if (!$packages) {

The expression $packages of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

                $this->cli->error('No packages available to install');
            }

            $handle = $this->confirmHandle($packages);
        }

        $this->cli->info('Installing ' . $handle);
        $this->bus->handle((new Install())->setHandle($handle));
        $this->cli->green('Installed Successfully!');
    }

    /**
     * Handle package:uninstall
     *
     * @param \Buttress\Concrete\Locator\Site $site
     */
    public function uninstall(Site $site)

This method seems to be duplicated in your project.

    {
        $definition = $this->getDefinition($site, 'package:uninstall');
        $definition->parse();
        $handle = $definition->get('handle');

        if (!$handle) {
            $packages = iterator_to_array($this->getAvailablePackages($site));
            if (!$packages) {

The expression $packages of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

                $this->cli->error('No packages available to install');
            }

            $handle = $this->confirmHandle($packages);
        }

        $this->cli->info('Installing ' . $handle);
        $this->bus->handle((new Uninstall())->setHandle($handle));
        $this->cli->green('Uninstalled Successfully!');
    }

    private function getDefinition(Site $site, $command)
    {
        /** @var \Buttress\Concrete\Console\Command\Manager\CommandManager[] $definitions */
        $definitions = $this->command->getCommands($site);
        foreach ($definitions as $definition) {
            if ($definition->getCommand() === $command) {
                return $definition;
            }
        }
    }

    private function getAvailablePackages(Site $site)
    {
        $iterator = new \DirectoryIterator($site->getPath() . '/packages');

        foreach ($iterator as $item) {
            if (!$item->isDot() && $item->isDir()) {
                yield $item->getBasename();
            }
        }
    }

    private function confirmHandle($packages)
    {
        $this->cli->green()->columns($packages)->br();

        $input = $this->cli->input('Which package?');
        $input->accept($packages);
        $input->strict();

        return $input->prompt();
    }

}