Shlinkio\Shlink\CLI\Command\Api\DisableKeyCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	configure()	0	5	1
A	execute()	0	12	2

<?php

declare(strict_types=1);

namespace Shlinkio\Shlink\CLI\Command\Api;

use Shlinkio\Shlink\CLI\Util\ExitCodes;
use Shlinkio\Shlink\Common\Exception\InvalidArgumentException;
use Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;

use function sprintf;

class DisableKeyCommand extends Command
{
    public const NAME = 'api-key:disable';

    private ApiKeyServiceInterface $apiKeyService;

    public function __construct(ApiKeyServiceInterface $apiKeyService)
    {
        parent::__construct();
        $this->apiKeyService = $apiKeyService;
    }

    protected function configure(): void
    {
        $this->setName(self::NAME)
             ->setDescription('Disables an API key.')
             ->addArgument('apiKey', InputArgument::REQUIRED, 'The API key to disable');
    }

    protected function execute(InputInterface $input, OutputInterface $output): ?int
    {
        $apiKey = $input->getArgument('apiKey');
        $io = new SymfonyStyle($input, $output);

        try {
            $this->apiKeyService->disable($apiKey);

It seems like $apiKey can also be of type null and string[]; however, parameter $key of Shlinkio\Shlink\Rest\Service\ApiKeyServiceInterface::disable() does only seem to accept string, maybe add an additional type check?

            $io->success(sprintf('API key "%s" properly disabled', $apiKey));

It seems like $apiKey can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

            return ExitCodes::EXIT_SUCCESS;
        } catch (InvalidArgumentException $e) {
            $io->error($e->getMessage());
            return ExitCodes::EXIT_FAILURE;
        }
    }
}