Terox\SubscriptionBundle\Command\AbstractCommand

Complexity

Total Complexity

4

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Test Coverage

Coverage

94.44%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getManager()	0	3	1
A	execute()	0	17	2
A	configure()	0	6	1

<?php

namespace Terox\SubscriptionBundle\Command;

use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Terox\SubscriptionBundle\Model\SubscriptionInterface;
use Terox\SubscriptionBundle\Subscription\SubscriptionManager;

abstract class AbstractCommand extends ContainerAwareCommand

The class Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand has been deprecated: since Symfony 4.2, use {@see Command} instead.

{
    /**
     * @var InputInterface
     */
    protected $input;

    /**
     * @var OutputInterface
     */
    protected $output;

    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->addArgument(
            'id',
            InputArgument::REQUIRED,
            'Subscription ID'
        );
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        // Save the input interfaces
        $this->input  = $input;
        $this->output = $output;

        $subscriptionId = $input->getArgument('id');
        $subscription   = $this->getContainer()->get('terox.subscription.repository.subscription')->findById($subscriptionId);

        if(null === $subscription) {
            return $output->writeln(sprintf('<error>The subscription with ID "%s" was not found.</error>', $subscriptionId));

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

        }

        // Execute the action
        $this->action($subscription);

        $output->writeln('<green>Finished.</green>');
    }

    /**
     * Action to execute when
     *
     * @param SubscriptionInterface $subscription
     *
     * @return void
     */
    abstract protected function action(SubscriptionInterface $subscription);

    /**
     * Get SubscriptionManager.
     *
     * @return SubscriptionManager
     */
    protected function getManager()
    {
        return $this->getContainer()->get('terox.subscription.manager');
    }
}