gggeek / db-3v4l

app/src/Command/SqlShell.php

<?php

Missing file doc comment

namespace Db3v4l\Command;

use Db3v4l\API\Interfaces\SqlExecutor\Forked\ShellExecutor;
use Db3v4l\Core\SqlExecutor\Forked\NativeClient;
use Db3v4l\Service\DatabaseConfigurationManager;
use Db3v4l\Service\SqlExecutorFactory;
use Symfony\Component\Console\Input\InputInterface;

The type Symfony\Component\Console\Input\InputInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Symfony\Component\Console\Input\InputOption;

The type Symfony\Component\Console\Input\InputOption was not found. Maybe you did not declare it correctly or list all dependencies?

use Symfony\Component\Console\Output\OutputInterface;

The type Symfony\Component\Console\Output\OutputInterface was not found. Maybe you did not declare it correctly or list all dependencies?

class SqlShell extends BaseCommand

Missing doc comment for class SqlShell

{
    protected static $defaultName = 'db3v4l:sql:shell';

    /** @var DatabaseConfigurationManager $dbConfigurationManager */

The open comment tag must be the only content on the line

Missing short description in doc comment

The close comment tag must be the only content on the line

    protected $dbConfigurationManager;
    /** @var SqlExecutorFactory $executorFactory */

The open comment tag must be the only content on the line

Missing short description in doc comment

The close comment tag must be the only content on the line

    protected $executorFactory;

    public function __construct(

Missing doc comment for function __construct()

        DatabaseConfigurationManager $dbConfigurationManager,
        SqlExecutorFactory $executorFactory)

The closing parenthesis of a multi-line function declaration must be on a new line

    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $this->dbConfigurationManager = $dbConfigurationManager;
        $this->executorFactory = $executorFactory;

        parent::__construct();
    }

    protected function configure()

Missing doc comment for function configure()

    {
        $this
            ->setDescription('Connects to one of the configured database instances, using the appropriate native sql client')
            ->addOption('instance', 'i', InputOption::VALUE_REQUIRED, 'The instance to connect to', null)
            ->addOption('database', 'd', InputOption::VALUE_REQUIRED, 'The name of an existing database to use')

Space after closing parenthesis of function call prohibited

        ;
    }

    protected function execute(InputInterface $input, OutputInterface $output)

Missing doc comment for function execute()

    {
        $instanceName = $input->getOption('instance');
        $dbName = $input->getOption('database');

        if ($instanceName == null) {
            throw new \Exception("Please provide an instance name");
        }

        $dbConnectionSpec = $this->dbConfigurationManager->getInstanceConfiguration($instanceName);
        if ($dbName != '') {
            $dbConnectionSpec['dbname'] = $dbName;
        }
        $executor = $this->executorFactory->createForkedExecutor($instanceName, $dbConnectionSpec, NativeClient::EXECUTION_STRATEGY, false);

        if (! $executor instanceof ShellExecutor) {
            throw new \Exception("Can not start an interactive shell for databases of type '{$dbConnectionSpec['vendor']}'");
        }

        $process = $executor->getExecuteShellProcess();

        $process->setTty(true);
        $process->run();

        return $process->getExitCode();
    }
}