gggeek / db-3v4l

app/admin/public/index.php

Spacing +2 added lines, -3 removed lines

@@ -7,12 +7,11 @@
     // Avoid bootstrapping Symfony for something so trivial...
     $config = \Symfony\Component\Yaml\Yaml::parseFile(dirname(dirname(__DIR__)).'/config/services.yaml');
     $servers = [];
-    foreach($config['parameters']['db3v4l.database_instances'] as $instance => $def) {
+    foreach ($config['parameters']['db3v4l.database_instances'] as $instance => $def) {
         $servers[$instance] = new AdminerLoginServerEnhanced(
             (
                 $def['vendor'] == 'oracle' ?
-                ('//'.$def['host'].':'.$def['port'].'/'.$def['servicename']) :
-                ($def['host'].':'.$def['port'])
+                ('//'.$def['host'].':'.$def['port'].'/'.$def['servicename']) : ($def['host'].':'.$def['port'])
             ),
             $def['vendor'].' '.$def['version'],
             str_replace(

app/src/Core/SqlExecutor/Forked/NativeClient.php

Spacing +26 added lines, -26 removed lines

@@ -54,10 +54,10 @@ 
             case 'mysql':
                 $command = 'mysql';
                 $options = [
-                    '--host=' . $this->databaseConfiguration['host'],
-                    '--port=' . $this->databaseConfiguration['port'] ?? '3306',
-                    '--user=' . $this->databaseConfiguration['user'],
-                    '-p' . $this->databaseConfiguration['password'],
+                    '--host='.$this->databaseConfiguration['host'],
+                    '--port='.$this->databaseConfiguration['port'] ?? '3306',
+                    '--user='.$this->databaseConfiguration['user'],
+                    '-p'.$this->databaseConfiguration['password'],
                     '--binary-mode', // 'It also disables all mysql commands except charset and delimiter in non-interactive mode (for input piped to mysql or loaded using the source command)'
                     '-t',
                 ];
@@ -65,7 +65,7 @@ 
                     $options[] = $this->databaseConfiguration['dbname'];
                 }
                 if ($action == self::EXECUTE_COMMAND) {
-                    $options[] = '--execute=' . $sqlOrFilename;
+                    $options[] = '--execute='.$sqlOrFilename;
                 }
                // $env = [
                     // problematic when wrapping the process in a call to `time`...
@@ -87,7 +87,7 @@ 
                 // NB: this triggers a different behaviour that piping multiple commands to stdin, namely
                 // it wraps all of the commands in a transaction and allows either sql commands or a single meta-command
                 if ($action == self::EXECUTE_COMMAND) {
-                    $options[] = '--command=' . $sqlOrFilename;
+                    $options[] = '--command='.$sqlOrFilename;
                 }
                 //$env = [
                     // problematic when wrapping the process in a call to `time`...
@@ -98,19 +98,19 @@ 
             case 'sqlcmd':
                 $command = 'sqlcmd';
                 $options = [
-                    '-S' . $this->databaseConfiguration['host'] . ($this->databaseConfiguration['port'] != '' ?  ',' . $this->databaseConfiguration['port'] : ''),
-                    '-U' . $this->databaseConfiguration['user'],
-                    '-P' . $this->databaseConfiguration['password'],
+                    '-S'.$this->databaseConfiguration['host'].($this->databaseConfiguration['port'] != '' ? ','.$this->databaseConfiguration['port'] : ''),
+                    '-U'.$this->databaseConfiguration['user'],
+                    '-P'.$this->databaseConfiguration['password'],
                     '-r1',
                     '-b',
                 ];
                 if (isset($this->databaseConfiguration['dbname'])) {
-                    $options[] = '-d' . $this->databaseConfiguration['dbname'];
+                    $options[] = '-d'.$this->databaseConfiguration['dbname'];
                 }
                 if ($action == self::EXECUTE_FILE) {
-                    $options[] = '-i' . $sqlOrFilename;
+                    $options[] = '-i'.$sqlOrFilename;
                 } elseif ($action == self::EXECUTE_COMMAND) {
-                    $options[] = '-Q' . $sqlOrFilename;
+                    $options[] = '-Q'.$sqlOrFilename;
                 }
                 break;
 
@@ -119,7 +119,7 @@ 
                 // 'path' is the full path to the 'master' db (for Doctrine compatibility).
                 //  non-master dbs are supposed to reside in the same directory
                 if (isset($this->databaseConfiguration['dbname'])) {
-                    $options[] = dirname($this->databaseConfiguration['path']) . '/' . $this->databaseConfiguration['dbname'] . '.sqlite';
+                    $options[] = dirname($this->databaseConfiguration['path']).'/'.$this->databaseConfiguration['dbname'].'.sqlite';
                 } else {
                     $options[] = $this->databaseConfiguration['path'];
                 }
@@ -133,22 +133,22 @@ 
                 /// @todo disable execution of dangerous (or all) SQLPLUS commands.
                 ///       See the list at https://docs.oracle.com/en/database/oracle/oracle-database/18/sqpug/SQL-Plus-command-reference.html#GUID-177F24B7-D154-4F8B-A05B-7568079800C6
                 $command = 'sqlplus';
-                $connectionIdentifier = '//' . $this->databaseConfiguration['host'] .
-                    ($this->databaseConfiguration['port'] != '' ?  ':' . $this->databaseConfiguration['port'] : '');
+                $connectionIdentifier = '//'.$this->databaseConfiguration['host'].
+                    ($this->databaseConfiguration['port'] != '' ? ':'.$this->databaseConfiguration['port'] : '');
                 // nb: for oracle, if we use pdbs to map 'databases', they get a new service name
                 /// @todo allow support for _not_ doing that, and using schemas as 'databases'
                 if (isset($this->databaseConfiguration['servicename'])) {
-                    $connectionIdentifier .= '/' . $this->databaseConfiguration['servicename'];
+                    $connectionIdentifier .= '/'.$this->databaseConfiguration['servicename'];
                 } else {
                     if ($this->databaseConfiguration['dbname'] != '') {
-                        $connectionIdentifier .= '/' . $this->databaseConfiguration['dbname'];
+                        $connectionIdentifier .= '/'.$this->databaseConfiguration['dbname'];
                     }
                 }
                 $options = [
                     '-L', // 'attempts to log in just once, instead of reprompting on error'
                     '-NOLOGINTIME',
                     '-S',
-                    $this->databaseConfiguration['user'] . '/' . $this->databaseConfiguration['password'] . '@' . $connectionIdentifier,
+                    $this->databaseConfiguration['user'].'/'.$this->databaseConfiguration['password'].'@'.$connectionIdentifier,
                 ];
                 /// @todo make this optional somehow / allow SYSOPER as alternative
                 if (strtolower($this->databaseConfiguration['user']) === 'sys') {
@@ -158,10 +158,10 @@ 
                 if ($action == self::EXECUTE_FILE) {
                     /// @todo wouldn't it be better to create another temp file with prefixed the sqlplus commands?
                     //$options[] = '@' . $sqlOrFilename;
-                    $sqlOrFilename = "WHENEVER OSERROR EXIT FAILURE;\nWHENEVER SQLERROR EXIT SQL.SQLCODE;\nSET PAGESIZE 50000;\nSET FEEDBACK OFF;\n" . file_get_contents($sqlOrFilename);
+                    $sqlOrFilename = "WHENEVER OSERROR EXIT FAILURE;\nWHENEVER SQLERROR EXIT SQL.SQLCODE;\nSET PAGESIZE 50000;\nSET FEEDBACK OFF;\n".file_get_contents($sqlOrFilename);
                     $action = self::EXECUTE_COMMAND;
                 } else {
-                    $sqlOrFilename = "WHENEVER OSERROR EXIT FAILURE;\nWHENEVER SQLERROR EXIT SQL.SQLCODE;\nSET PAGESIZE 50000;\nSET FEEDBACK OFF;\n" . $sqlOrFilename;
+                    $sqlOrFilename = "WHENEVER OSERROR EXIT FAILURE;\nWHENEVER SQLERROR EXIT SQL.SQLCODE;\nSET PAGESIZE 50000;\nSET FEEDBACK OFF;\n".$sqlOrFilename;
                 }
 
                 break;
@@ -174,11 +174,11 @@ 
 
         /// @todo investigate: for psql is this better done via --file ?
         if ($action == self::EXECUTE_FILE && $clientType != 'sqlsrv' && $clientType != 'sqlplus') {
-            $commandLine .= ' < ' . escapeshellarg($sqlOrFilename);
+            $commandLine .= ' < '.escapeshellarg($sqlOrFilename);
         }
 
         if ($action == self::EXECUTE_COMMAND && $clientType == 'sqlplus') {
-            $commandLine .= " << 'SQLEOF'\n" . $sqlOrFilename . "\nSQLEOF";
+            $commandLine .= " << 'SQLEOF'\n".$sqlOrFilename."\nSQLEOF";
         }
 
         $process = Process::fromShellCommandline($commandLine, null, $env);
@@ -225,7 +225,7 @@ 
                 array_shift($output); // headers
                 array_shift($output); // '+--+'
                 array_pop($output); // '+--+'
-                foreach($output as &$line) {
+                foreach ($output as &$line) {
                     $line = trim($line, '|');
                     $line = trim($line);
                 }
@@ -235,7 +235,7 @@ 
                 array_shift($output); // empty line
                 array_shift($output); // headers
                 array_shift($output); // '---'
-                foreach($output as &$line) {
+                foreach ($output as &$line) {
                     $line = trim($line);
                 }
                 return $output;
@@ -244,7 +244,7 @@ 
                 array_shift($output); // headers
                 array_shift($output); // '---'
                 //array_pop($output); // '(N rows)'
-                foreach($output as &$line) {
+                foreach ($output as &$line) {
                     $line = trim($line);
                 }
                 return $output;
@@ -257,7 +257,7 @@ 
                 array_shift($output); // '---'
                 array_pop($output); // blank line
                 array_pop($output); // '(N rows affected)'
-                foreach($output as &$line) {
+                foreach ($output as &$line) {
                     $line = trim($line);
                 }
                 return $output;

app/src/Core/DatabaseManager/PostgreSQL.php

Spacing +6 added lines, -6 removed lines

@@ -17,7 +17,7 @@ 
     {
         return new Command(
             'SELECT datname AS "Database" FROM pg_database ORDER BY datname;',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -40,11 +40,11 @@ 
         $statements = [
             // q: do we need to add 'TEMPLATE template0' ?
             //    see f.e. https://www.vertabelo.com/blog/collations-in-postgresql/
-            "CREATE DATABASE \"$dbName\"" . ($collation !== null ? " ENCODING $collation" : '') . ';',
+            "CREATE DATABASE \"$dbName\"".($collation !== null ? " ENCODING $collation" : '').';',
         ];
         if ($userName != '') {
             $statements[] = "COMMIT;";
-            $statements[] = "CREATE USER \"$userName\" WITH PASSWORD '$password'" . ';';
+            $statements[] = "CREATE USER \"$userName\" WITH PASSWORD '$password'".';';
             $statements[] = "GRANT ALL ON DATABASE \"$dbName\" TO \"$userName\""; // q: should we avoid granting CREATE?
         }
         return new Command($statements);
@@ -82,7 +82,7 @@ 
     {
         return new Command(
             'SELECT usename AS "User" FROM pg_catalog.pg_user ORDER BY usename;',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -115,7 +115,7 @@ 
     {
         return new Command(
             'SELECT collname AS Collation FROM pg_collation ORDER BY collname',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -130,7 +130,7 @@ 
     {
         return new Command(
             'SHOW server_version;',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output)[0];
             }

app/src/Core/DatabaseManager/SQLServer.php

Spacing +6 added lines, -6 removed lines

@@ -19,7 +19,7 @@ 
         // the way we create it, the user account is contained in the db
         // @todo add "WHERE name NOT IN ('master', 'tempdb', 'model', 'msdb')" ?
             "SELECT name AS 'Database' FROM sys.databases ORDER BY name;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -45,7 +45,7 @@ 
             // Also, by default connections are in autocommit mode...
             // And yet, we need a GO to commit the db creation...
             "SET QUOTED_IDENTIFIER ON;",
-            "CREATE DATABASE \"$dbName\"" . ($collation !== null ? " COLLATE $collation" : '') . ';'
+            "CREATE DATABASE \"$dbName\"".($collation !== null ? " COLLATE $collation" : '').';'
         ];
         if ($userName != '') {
             $statements[] = "CREATE LOGIN \"$userName\" WITH PASSWORD = '$password', DEFAULT_DATABASE = \"$dbName\", CHECK_POLICY = OFF, CHECK_EXPIRATION = OFF;";
@@ -95,7 +95,7 @@ 
     {
         return new Command(
             "SELECT name AS 'User' FROM sys.sql_logins ORDER BY name",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -132,7 +132,7 @@ 
     {
         return new Command(
             'SELECT name AS Collation FROM fn_helpcollations();',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -147,12 +147,12 @@ 
     {
         return new Command(
             "SELECT @@version",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 $output = $executor->resultSetToArray($output);
                 $line = $output[0];
                 preg_match('/Microsoft SQL Server +([^ ]+) +([^ ]+) +/', $line, $matches);
-                return $matches[1] . ' ' . $matches[2];
+                return $matches[1].' '.$matches[2];
             }
         );
     }

app/src/Core/DatabaseManager/MySQL.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
         return new Command(
         /// @todo use 'SHOW DATABASES' for versions < 5
             "SELECT SCHEMA_NAME AS 'Database' FROM information_schema.SCHEMATA ORDER BY SCHEMA_NAME;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -40,7 +40,7 @@ 
 
         /// @todo if mysql version is bigger than 8.0, add `WITH mysql_native_password`
         $statements = [
-            "CREATE DATABASE `$dbName`" . ($collation !== null ? " CHARACTER SET $collation" : '') . ';'
+            "CREATE DATABASE `$dbName`".($collation !== null ? " CHARACTER SET $collation" : '').';'
         ];
         if ($userName != '') {
             $statements[] = "CREATE USER '$userName'@'%' IDENTIFIED BY '$password';";
@@ -83,7 +83,7 @@ 
     {
         return new Command(
             'SELECT DISTINCT User FROM mysql.user ORDER BY User;',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -118,13 +118,13 @@ 
     {
         return new Command(
             'SHOW COLLATION;',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 $lines = $executor->resultSetToArray($output);
                 $out = [];
-                foreach($lines as $line) {
+                foreach ($lines as $line) {
                     $parts = explode("|", $line, 3);
-                    $out[] = trim($parts[0]) . ' (' . trim($parts[1]) .')';
+                    $out[] = trim($parts[0]).' ('.trim($parts[1]).')';
                 }
                 return $out;
             }
@@ -139,7 +139,7 @@ 
     {
         return new Command(
             'SHOW VARIABLES LIKE "version";',
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 $line = $executor->resultSetToArray($output)[0];
                 $parts = explode('|', $line);

app/src/Core/DatabaseManager/SQLite.php

Spacing +7 added lines, -7 removed lines

@@ -15,13 +15,13 @@ 
      */
     public function getListDatabasesSqlAction()
     {
-        $fileGlob = dirname($this->databaseConfiguration['path']) . '/*.sqlite';
+        $fileGlob = dirname($this->databaseConfiguration['path']).'/*.sqlite';
         return new Command(
             null,
             function() use ($fileGlob) {
                 $out = [];
                 foreach (glob($fileGlob) as $filename) {
-                    $out[] =  basename($filename);
+                    $out[] = basename($filename);
                 }
                 return $out;
             }
@@ -45,7 +45,7 @@ 
 
         /// @todo this does not support creation of the new db with a different character encoding...
         ///       see https://stackoverflow.com/questions/21348459/set-pragma-encoding-utf-16-for-main-database-in-sqlite
-        $filename = dirname($this->databaseConfiguration['path']) . '/' . $dbName . '.sqlite';
+        $filename = dirname($this->databaseConfiguration['path']).'/'.$dbName.'.sqlite';
         return new Command(
             "ATTACH '$filename' AS \"$dbName\";"
         );
@@ -61,7 +61,7 @@ 
      */
     public function getDropDatabaseSqlAction($dbName, $userName, $ifExists = false)
     {
-        $filename = dirname($this->databaseConfiguration['path']) . '/' . $dbName . '.sqlite';
+        $filename = dirname($this->databaseConfiguration['path']).'/'.$dbName.'.sqlite';
         return new Command(
             null,
             function() use($filename, $dbName, $ifExists) {
@@ -84,7 +84,7 @@ 
     {
         return new Command(
             null,
-            function () {
+            function() {
                 // since sqlite does not support users, null seems more appropriate than an empty array...
                 return null;
             }
@@ -116,7 +116,7 @@ 
         return new Command(
             null,
             /// @todo list the supported utf16 variants as soon as allow using them
-            function () {
+            function() {
                 return [];
             }
         );
@@ -142,7 +142,7 @@ 
     {
         return new Command(
             "select sqlite_version();",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output)[0];
             }

app/src/Core/DatabaseManager/OraclePDB.php

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
     {
         return new Command(
             "SELECT pdb_name AS Database FROM dba_pdbs ORDER BY pdb_name;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -91,7 +91,7 @@ 
         return new Command(
             // NB: we filter out 'system' users, as there are many...
             "SELECT username FROM cdb_users WHERE oracle_maintained != 'Y' ORDER BY username;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -122,7 +122,7 @@ 
     {
         return new Command(
             "SELECT value AS Collation FROM v\$nls_valid_values WHERE parameter = 'CHARACTERSET' ORDER BY value;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -137,7 +137,7 @@ 
     {
         return new Command(
             "SELECT version_full || ' (' || banner || ')' AS version FROM  v\$version, v\$instance;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output)[0];
             }

app/src/Core/DatabaseManager/Oracle.php

Spacing +4 added lines, -4 removed lines

@@ -22,7 +22,7 @@ 
     {
         return new Command(
             "SELECT username AS Database FROM all_users WHERE oracle_maintained != 'Y' ORDER BY username;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -92,7 +92,7 @@ 
         return new Command(
             // NB: we filter out 'system' users, as there are many...
             "SELECT username FROM sys.all_users WHERE oracle_maintained != 'Y' ORDER BY username;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -124,7 +124,7 @@ 
     {
         return new Command(
             "SELECT value AS Collation FROM v\$nls_valid_values WHERE parameter = 'CHARACTERSET' ORDER BY value;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output);
             }
@@ -139,7 +139,7 @@ 
     {
         return new Command(
             "SELECT version_full || ' (' || banner || ')' AS version FROM  v\$version, v\$instance;",
-            function ($output, $executor) {
+            function($output, $executor) {
                 /** @var Executor $executor */
                 return $executor->resultSetToArray($output)[0];
             }

app/src/Command/SQLExecutingCommand.php

Spacing +13 added lines, -13 removed lines

@@ -55,8 +55,8 @@ 
     protected function addCommonOptions()
     {
         $this
-            ->addOption('only-instances', 'o', InputOption::VALUE_REQUIRED|InputOption::VALUE_IS_ARRAY, 'Filter the database servers to run this command against. Usage of * and ? wildcards is allowed. To see all instances available, use `instance:list`', null)
-            ->addOption('except-instances', 'x', InputOption::VALUE_REQUIRED|InputOption::VALUE_IS_ARRAY, 'Filter the database servers to run this command against', null)
+            ->addOption('only-instances', 'o', InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, 'Filter the database servers to run this command against. Usage of * and ? wildcards is allowed. To see all instances available, use `instance:list`', null)
+            ->addOption('except-instances', 'x', InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, 'Filter the database servers to run this command against', null)
             ->addOption('output-type', null, InputOption::VALUE_REQUIRED, 'The format for the output: json, php, text or yml', self::DEFAULT_OUTPUT_FORMAT)
             ->addOption('output-file', null, InputOption::VALUE_REQUIRED, 'Save output to a file instead of writing it to stdout. NB: take care that dbconsole runs in a container, which has a different view of the filesystem. A good dir for output is ./shared')
             ->addOption('timeout', null, InputOption::VALUE_REQUIRED, 'The maximum time to wait for subprocess execution (secs)', self::DEFAULT_PROCESS_TIMEOUT)
@@ -83,7 +83,7 @@ 
         // On Debian, which we use by default, SF has troubles understanding that php was compiled with --enable-sigchild
         // We thus force it, but give end users an option to disable this
         // For more details, see comment 12 at https://bugs.launchpad.net/ubuntu/+source/php5/+bug/516061
-        if (! $input->getOption('dont-force-enabled-sigchild')) {
+        if (!$input->getOption('dont-force-enabled-sigchild')) {
             Process::forceSigchildEnabled(true);
         }
 
@@ -127,7 +127,7 @@ 
                     $sql = null;
                 } else {
                     // this is a coding error, not a sql execution error
-                    throw new \Exception("Unsupported action type: " . get_class($sqlAction));
+                    throw new \Exception("Unsupported action type: ".get_class($sqlAction));
                 }
                 $filterCallable = $sqlAction->getResultsFilterCallable();
 
@@ -138,7 +138,7 @@ 
                     $outputFilters[$instanceName] = $filterCallable;
 
                     if ($filename === null && !$sqlAction->isSingleStatement()) {
-                        $filename = tempnam(sys_get_temp_dir(), 'db3v4l_') . '.sql';
+                        $filename = tempnam(sys_get_temp_dir(), 'db3v4l_').'.sql';
                         file_put_contents($filename, $sql);
                         $tempSQLFileNames[] = $filename;
                     }
@@ -161,7 +161,7 @@ 
                         }
 
                         if ($this->outputFormat === 'text') {
-                            $this->writeln('Command line: ' . $process->getCommandLine(), OutputInterface::VERBOSITY_VERY_VERBOSE);
+                            $this->writeln('Command line: '.$process->getCommandLine(), OutputInterface::VERBOSITY_VERY_VERBOSE);
                         }
 
                         $process->setTimeout($this->processTimeout);
@@ -187,7 +187,7 @@ 
                         'stderr' => $t->getMessage(),
                     ];
                     $failed++;
-                    $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: " . $t->getMessage() . "</error>\n", OutputInterface::VERBOSITY_NORMAL);
+                    $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: ".$t->getMessage()."</error>\n", OutputInterface::VERBOSITY_NORMAL);
                 }
             }
 
@@ -212,7 +212,7 @@ 
                                 ];
                                 $failed++;
                                 $succeeded--;
-                                $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: " . $t->getMessage() . "</error>\n", OutputInterface::VERBOSITY_NORMAL);
+                                $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: ".$t->getMessage()."</error>\n", OutputInterface::VERBOSITY_NORMAL);
                             }
                         }
                         $results[$instanceName] = $output;
@@ -229,19 +229,19 @@ 
                         ];
 
                         $failed++;
-                        $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: " . $err . "</error>\n", OutputInterface::VERBOSITY_NORMAL);
+                        $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: ".$err."</error>\n", OutputInterface::VERBOSITY_NORMAL);
                     }
                 }
             }
 
         } finally {
             // make sure that we clean up temp files, as they might contain sensitive data
-            foreach($tempSQLFileNames as $tempSQLFileName) {
+            foreach ($tempSQLFileNames as $tempSQLFileName) {
                 unlink($tempSQLFileName);
             }
         }
 
-        uksort($results, function ($a, $b) {
+        uksort($results, function($a, $b) {
             $aParts = explode('_', $a, 2);
             $bParts = explode('_', $b, 2);
             $cmp = strcasecmp($aParts[0], $bParts[0]);
@@ -300,11 +300,11 @@ 
             $this->writeResultsToFile($results);
         } else {
             $formattedResults = $this->formatResults($results);
-            $this->writeln($formattedResults, OutputInterface::VERBOSITY_QUIET,  OutputInterface::OUTPUT_RAW);
+            $this->writeln($formattedResults, OutputInterface::VERBOSITY_QUIET, OutputInterface::OUTPUT_RAW);
         }
 
         if ($this->outputFormat === 'text' || $this->outputFile != null) {
-            $this->writeln($results['succeeded'] . ' succeeded, ' . $results['failed'] . ' failed');
+            $this->writeln($results['succeeded'].' succeeded, '.$results['failed'].' failed');
             if ($this->outputFile != null) {
                 $this->writeln("Results saved to file {$this->outputFile}");
             }