AppBundle\ShowUnusedMySQLTables\Command

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	4	1
A	execute()	0	8	1

<?php

namespace AppBundle\ShowUnusedMySQLTables;

use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle;

/**
 * Symfony-Console-Command wrapper for the ShowUnusedMySQLTables task.
 */
final class Command extends ContainerAwareCommand
{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->setName('show-unused-mysql-tables')
             ->setDescription('Show a list of potentially unused MySQL tables.');
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $container = $this->getContainer();
        $task = new Task(
            $container->get('doctrine.dbal.default_connection'),

It seems like $container->get('doctrine.dbal.default_connection') can also be of type null; however, parameter $connection of AppBundle\ShowUnusedMySQLTables\Task::__construct() does only seem to accept Doctrine\DBAL\Connection, maybe add an additional type check?

            $container->getParameter('db_system_catalog_prefix')
        );
        $task->getUnusedTableNames(new SymfonyStyle($input, $output));
    }
}