gggeek / db-3v4l

app/src/Command/SqlShell.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
         }
         $executor = $this->executorFactory->createForkedExecutor($dbConnectionSpec, 'NativeClient', false);
 
-        if (! $executor instanceof ShellExecutor) {
+        if (!$executor instanceof ShellExecutor) {
             throw new \Exception("Can not start an interactive shell for databases of type '{$dbConnectionSpec['vendor']}'");
         }
 

app/src/Command/SQLExecutingCommand.php

Spacing +10 added lines, -10 removed lines

@@ -70,7 +70,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);
         }
 
@@ -114,7 +114,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();
 
@@ -129,7 +129,7 @@ 
 
                     if ($filename === null) {
                         if (!$sqlAction->isSingleStatement()) {
-                            $tempSQLFileName = tempnam(sys_get_temp_dir(), 'db3v4l_') . '.sql';
+                            $tempSQLFileName = tempnam(sys_get_temp_dir(), 'db3v4l_').'.sql';
                             file_put_contents($tempSQLFileName, $sql);
                             $tempSQLFileNames[] = $tempSQLFileName;
 
@@ -142,7 +142,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);
@@ -161,7 +161,7 @@ 
                     $succeeded++;
                 } catch (\Throwable $t) {
                     $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);
                 }
             }
 
@@ -183,7 +183,7 @@ 
                                 //$result = null;
                                 $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;
@@ -194,14 +194,14 @@ 
                             'exitcode' => $process->getExitCode()
                         ];
                         $failed++;
-                        $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: " . $process->getErrorOutput() . "</error>\n", OutputInterface::VERBOSITY_NORMAL);
+                        $this->writeErrorln("\n<error>$actionName in instance '$instanceName' failed! Reason: ".$process->getErrorOutput()."</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);
             }
         }
@@ -250,12 +250,12 @@ 
         if ($this->outputFile != null) {
             file_put_contents($this->outputFile, $formattedResults);
         } else {
-            $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}");
             }

app/src/Command/InstanceList.php

Spacing +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
         $result = $this->listInstances($instanceList);
 
         $extraResults = $this->listDatabasesVersion($instanceList)['data'];
-        foreach($result as $instanceName => $instanceDesc) {
+        foreach ($result as $instanceName => $instanceDesc) {
             if (isset($extraResults[$instanceName])) {
                 $result[$instanceName]['version'] = $extraResults[$instanceName];
             }
@@ -56,7 +56,7 @@ 
     protected function listInstances($instanceList)
     {
         $out = [];
-        foreach($instanceList as $instanceName => $connectionSpec) {
+        foreach ($instanceList as $instanceName => $connectionSpec) {
             //$connectionSpec = $this->dbConfigurationManager->getInstanceConfiguration($instanceName);
             $out[$instanceName] = [
                 'vendor' => $connectionSpec['vendor'],
@@ -71,7 +71,7 @@ 
         return $this->executeSqlAction(
             $instanceList,
             'Getting database version information',
-            function ($schemaManager, $instanceName) {
+            function($schemaManager, $instanceName) {
                 /** @var DatabaseSchemaManager $schemaManager */
                 return $schemaManager->getRetrieveVersionInfoSqlAction();
             }
@@ -101,7 +101,7 @@ 
             file_put_contents($this->outputFile, $data);
             $this->writeln("Results saved to file {$this->outputFile}");
         } else {
-            $this->writeln($data, OutputInterface::VERBOSITY_QUIET,  OutputInterface::OUTPUT_RAW);
+            $this->writeln($data, OutputInterface::VERBOSITY_QUIET, OutputInterface::OUTPUT_RAW);
         }
 
     }