Db3v4l\Command\SqlExecute::execute()

Complexity

Conditions	19
Paths	111

Size

Total Lines	131
Code Lines	70

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	1

<?php
    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);

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

        if ($sql == null && $file == null) {
            throw new \Exception("Please provide an sql command or file to be executed");
        }
        if ($sql != null && $file != null) {
            throw new \Exception("Please provide either an sql command or file to be executed, not both");
        }

        if (($dbName == null) xor ($userName == null)) {

            /// @todo is it possible to let the user log in to the 'root' db, regardless of db type ?

            throw new \Exception("Please provide both a custom database name and associated user account");
        }

        $createDB = ($dbName == null);

        if ($createDB) {
            // create temp databases

            if ($this->outputFormat === 'text') {
                $this->writeln('<info>Creating temporary databases...</info>', OutputInterface::VERBOSITY_VERBOSE);
            }

            /// @todo inject more randomness in the username, by allowing more chars than bin2hex produces

            $userName = 'db3v4l_' . substr(bin2hex(random_bytes(5)), 0, 9); // some mysql versions have a limitation of 16 chars for usernames
            $password = bin2hex(random_bytes(15)); // some oracle versions have a limit of 30 chars on passwords
            //$dbName = bin2hex(random_bytes(31));
            $dbName = $userName; // $userName will be used as db name

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

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

                $tempDbSpecs[$instanceName] = [
                    'dbname' => $dbName,
                    'user' => $userName,
                    'password' => $password
                ];
            }

            if (($charset = $input->getOption('charset')) != '') {
                foreach($instanceList as $instanceName) {

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

                    $tempDbSpecs[$instanceName]['charset'] = $charset;
                }
            }

            // for now, we always use NativeClient for creating/dropping temp dbs
            $previousStrategy = $this->executionStrategy;
            $this->executionStrategy = NativeClient::EXECUTION_STRATEGY;
            $creationResults = $this->createDatabases($instanceList, $tempDbSpecs);
            $this->executionStrategy = $previousStrategy;

            $dbConnectionSpecs = $creationResults['data'];

        } else {
            // use existing databases

            if ($this->outputFormat === 'text') {
                $this->writeln('<info>Using existing databases...</info>', OutputInterface::VERBOSITY_VERBOSE);
            }

            $dbConfig = [
                'user' => $userName,
                'password' => $password,
                'dbname' => $dbName,
            ];

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

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

                $dbConnectionSpecs[$instanceName] = array_merge($instanceSpecs, $dbConfig);
            }
        }

        if (count($dbConnectionSpecs)) {

            if ($this->outputFormat === 'text') {
                $this->writeln('<info>Executing sql command/file...</info>', OutputInterface::VERBOSITY_VERBOSE);
            }

            $results = $this->executeSQL($dbConnectionSpecs, $sql, $file);

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

            if ($createDB) {
                $results['failed'] += $creationResults['failed'];

The variable $creationResults does not seem to be defined for all execution paths leading up to this point.

                foreach($instanceList as $instanceName  => $instanceSpecs) {

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

                    if (!isset($dbConnectionSpecs[$instanceName])) {
                        /// @todo retrieve the actual temp db creation error instead of doing this...
                        $results['data'][$instanceName] = [
                            'exitcode' => '1',
                            'stderr' => 'Error in creation of temporary database'
                        ];
                        unset($instanceList[$instanceName]);
                    }
                }
                // for now, we always use NativeClient for creating/dropping temp dbs
                $previousStrategy = $this->executionStrategy;
                $this->executionStrategy = NativeClient::EXECUTION_STRATEGY;
                $this->dropDatabases($instanceList, $dbConnectionSpecs, true);
                $this->executionStrategy = $previousStrategy;
            }
        } else {
            $results = ['succeeded' => 0,  'failed' => 0, 'data' => null];
        }

        $time = microtime(true) - $start;

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

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