T3G\Elasticorn\Commands\Mapping\CompareCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	10	1
A	execute()	0	5	1

<?php
declare(strict_types = 1);
namespace T3G\Elasticorn\Commands\Mapping;


use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use T3G\Elasticorn\Commands\BaseCommand;

/**
 * @package T3G\Elasticorn\Commands
 */
class CompareCommand extends BaseCommand
{
    /**
     * Configure the compare command
     *
     * @return void
     */
    protected function configure()
    {
        parent::configure();
        $this
            ->setName('mapping:compare')
            ->setDescription('compare mapping configuration to applied configuration.');

        $this->
            addArgument('indexName', InputArgument::REQUIRED, 'The index name.');
    }

    /**
     * @param InputInterface $input
     * @param OutputInterface $output
     * @return void
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        parent::execute($input, $output);
        $this->configurationService->compareMappingConfiguration($input->getArgument('indexName'), $this->indexService->getIndex());

It seems like $input->getArgument('indexName') targeting Symfony\Component\Console\Input\InputInterface::getArgument() can also be of type array<integer,string> or null; however, T3G\Elasticorn\Service\ConfigurationService::compareMappingConfiguration() does only seem to accept string, maybe add an additional type check?

    }

}