Terox\SubscriptionBundle\Command\ExpireCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	action()	0	7	1
A	configure()	0	12	1

<?php

namespace Terox\SubscriptionBundle\Command;

use Symfony\Component\Console\Input\InputArgument;
use Terox\SubscriptionBundle\Model\SubscriptionInterface;
use Terox\SubscriptionBundle\TeroxSubscriptionBundle;

class ExpireCommand extends AbstractCommand
{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        parent::configure();

        $this
            ->setName(TeroxSubscriptionBundle::COMMAND_NAMESPACE.':expire')
            ->setDescription('Expire a subscription')
            ->addArgument(
                'reason',
                InputArgument::OPTIONAL,
                'Reason of expiration',
                'expired'
            );
    }

    /**
     * {@inheritdoc}
     */
    protected function action(SubscriptionInterface $subscription)
    {
        $reason = $this->input->getArgument('reason');

        $this->getManager()->expire($subscription, $reason);

It seems like $reason can also be of type string[]; however, parameter $reason of Terox\SubscriptionBundle\Subscription\SubscriptionManager::expire() does only seem to accept string, maybe add an additional type check?

        $this->output->writeln(sprintf('Expired subscription'));
    }
}