gggeek / db-3v4l

app/src/Command/DatabaseDrop.php

<?php

Missing file doc comment

namespace Db3v4l\Command;

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 DatabaseDrop extends DatabaseManagingCommand

Missing doc comment for class DatabaseDrop

{
    protected static $defaultName = 'db3v4l:database:drop';

    protected function configure()

Missing doc comment for function configure()

    {
        $this
            ->setDescription('Drops a database & associated user in parallel on all configured database instances')
            ->addOption('database', 'd', InputOption::VALUE_REQUIRED, 'The name of the database to drop')
            ->addOption('user', 'u', InputOption::VALUE_REQUIRED, 'The name of a user to drop. If omitted, no user will be dropped')
            ->addCommonOptions()

Space after closing parenthesis of function call prohibited

        ;
    }

    /**

Missing short description in doc comment

     * @param InputInterface $input

Missing parameter comment

Expected 2 spaces after parameter type; 1 found

Tag value for @param tag indented incorrectly; expected 2 spaces but found 1

     * @param OutputInterface $output

Missing parameter comment

Tag value for @param tag indented incorrectly; expected 2 spaces but found 1

     * @return int

Tag @return cannot be grouped with parameter tags in a doc comment

     * @throws \Exception

Tag @throws cannot be grouped with parameter tags in a doc comment

     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $start = microtime(true);

        // as per https://www.php.net/manual/en/function.ignore-user-abort.php: for cli scripts, it is probably a good idea
        // to use ignore_user_abort
        ignore_user_abort(true);

        $this->setOutput($output);
        $this->setVerbosity($output->getVerbosity());

        $instanceList = $this->parseCommonOptions($input);

        $userName = $input->getOption('user');
        $dbName = $input->getOption('database');

        // BC api
        if ($dbName == null && $userName != null) {
            $dbName = $userName;
        }

        if ($dbName == null) {
            throw new \Exception("Please provide a database name");
        }

        if ($this->outputFormat === 'text') {
            $this->writeln('<info>Dropping databases...</info>');
        }

        $dbToDropSpecs = [];
        foreach($instanceList as $instanceName => $instanceSpecs) {

Expected "foreach (...) {\n"; found "foreach(...) {\n"

            $dbToDropSpecs[$instanceName] = [
                'dbname' => $dbName
            ];
            if ($userName != null) {
                $dbToDropSpecs[$instanceName]['user'] = $userName;
            }
        }

        $results = $this->dropDatabases($instanceList, $dbToDropSpecs);

        // q: shall we print something more useful?
        $results['data'] = null;

        $time = microtime(true) - $start;

        $this->writeResults($results, $time);

        return (int)$results['failed'];
    }
}