LCI\MODX\Orchestrator\Console\Command\UninstallPackages

Complexity

Total Complexity

3

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	18	1
A	execute()	0	11	2

<?php

namespace LCI\MODX\Orchestrator\Console\Command;

use LCI\MODX\Console\Command\BaseCommand;
use LCI\MODX\Orchestrator\Orchestrator;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

class UninstallPackages extends BaseCommand
{
    public $loadMODX = true;

    protected function configure()
    {
        $this
            ->setName('orchestrator:remove')
            ->setAliases(['orch-remove'])
            ->setDescription('Uninstall Package, this will run migrations down for related packages');

        $this->addArgument(
            'package',
            InputArgument::REQUIRED,
            'Enter a valid package name, like lci/stockpile. Multiple packages can separated by a comma.'
            )
            ->addOption(
            'type',
            't',
            InputOption::VALUE_OPTIONAL,
            'Server type passed to run migrations, default is master. Possible master, staging, dev and local',
            'master'
            );
    }

    /**
     * Runs the command.
     *
     * @param InputInterface $input
     * @param OutputInterface $output
     * @return int|null|void
     * @throws \LCI\Blend\Exception\MigratorException
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $packages = explode(',', $input->getArgument('package'));

It seems like $input->getArgument('package') can also be of type string[]; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

        $type = $input->getOption('type');

        foreach ($packages as $package) {
            $output->writeln('### Orchestrator::uninstallComposerPackage() for '.$package.' and type: '.$type.'  ###');
            Orchestrator::uninstallComposerPackage($package, $type);

It seems like $type can also be of type string[]; however, parameter $type of LCI\MODX\Orchestrator\Orchestrator::uninstallComposerPackage() does only seem to accept string, maybe add an additional type check?

        }

        $output->writeln($this->getRunStats());
    }
}