Incapsula\Command\SubAccountDeleteCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	5	1
A	execute()	0	3	1
A	configure()	0	7	1

<?php

namespace Incapsula\Command;

use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class SubAccountDeleteCommand extends AbstractCommand
{
    /**
     * @var string
     */
    private $accountID;

    protected function configure()
    {
        parent::configure();

        $this
            ->setName('subaccount:delete')
            ->addArgument('account-id', InputArgument::REQUIRED, 'sub account identifier')
        ;
    }

    /**
     * @param InputInterface  $input
     * @param OutputInterface $output
     */
    protected function initialize(InputInterface $input, OutputInterface $output)
    {
        parent::initialize($input, $output);

        $this->accountID = $input->getArgument('account-id');

It seems like $input->getArgument('account-id') can also be of type string[]. However, the property $accountID is declared as type string. Maybe add an additional type check?

    }

    /**
     * @param InputInterface  $input
     * @param OutputInterface $output
     *
     * @return int
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->client->accounts()->delete($this->accountID);
    }
}