gggeek / db-3v4l

app/src/Command/CollationList.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 
         $this->writeResults($results, $time);
 
-        return (int)$results['failed'];
+        return (int) $results['failed'];
     }
 
     /**
@@ -60,7 +60,7 @@ 
         return $this->executeSqlAction(
             $instanceList,
             'Listing of collations',
-            function ($schemaManager, $instanceName) {
+            function($schemaManager, $instanceName) {
                 /** @var DatabaseSchemaManager $schemaManager */
                 return $schemaManager->getListCollationsSqlAction();
             }

app/src/Command/DatabaseList.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 
         $this->writeResults($results, $time);
 
-        return (int)$results['failed'];
+        return (int) $results['failed'];
     }
 
     /**
@@ -60,7 +60,7 @@ 
         return $this->executeSqlAction(
             $instanceList,
             'Listing of databases',
-            function ($schemaManager, $instanceName) {
+            function($schemaManager, $instanceName) {
                 /** @var DatabaseSchemaManager $schemaManager */
                 return $schemaManager->getListDatabasesSqlAction();
             }

app/src/Command/UserDrop.php

Spacing +3 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
         }
 
         $userToDropSpecs = [];
-        foreach($instanceList as $instanceName => $instanceSpecs) {
+        foreach ($instanceList as $instanceName => $instanceSpecs) {
             $userToDropSpecs[$instanceName] = [
                 'user' => $userName
             ];
@@ -64,7 +64,7 @@ 
 
         $this->writeResults($results, $time);
 
-        return (int)$results['failed'];
+        return (int) $results['failed'];
     }
 
     /**
@@ -79,7 +79,7 @@ 
         return $this->executeSqlAction(
             $instanceList,
             'Dropping of user',
-            function ($schemaManager, $instanceName) use ($userToDropSpecs, $ifExists) {
+            function($schemaManager, $instanceName) use ($userToDropSpecs, $ifExists) {
                 $dbConnectionSpec = $userToDropSpecs[$instanceName];
                 /** @var DatabaseSchemaManager $schemaManager */
                 return $schemaManager->getDropUserSqlAction(

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);
         }
 
     }

app/src/Service/DatabaseConfigurationManager.php

Spacing +4 added lines, -4 removed lines

@@ -29,13 +29,13 @@ 
         }
 
         $names = [];
-        foreach(array_keys($this->instanceList) as $name) {
+        foreach (array_keys($this->instanceList) as $name) {
 
             if (empty($includeFilter)) {
                 $include = true;
             } else {
                 $include = false;
-                foreach($includeFilter as $filter) {
+                foreach ($includeFilter as $filter) {
                     if (fnmatch($filter, $name)) {
                         $include = true;
                         break;
@@ -44,7 +44,7 @@ 
             }
 
             if ($include && !empty($excludeFilter)) {
-                foreach($excludeFilter as $filter) {
+                foreach ($excludeFilter as $filter) {
                     if (fnmatch($filter, $name)) {
                         $include = false;
                         break;
@@ -81,7 +81,7 @@ 
     public function getInstancesConfiguration(array $instanceNames)
     {
         $instancesDefinitions = [];
-        foreach($instanceNames as $instanceName)
+        foreach ($instanceNames as $instanceName)
         {
             $instancesDefinitions[$instanceName] = $this->getInstanceConfiguration($instanceName);
         }

app/src/Command/SqlShell.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
         }
         $executor = $this->executorFactory->createForkedExecutor($instanceName, $dbConnectionSpec, NativeClient::EXECUTION_STRATEGY, 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/Core/SqlExecutor/Forked/PDO.php

Spacing +7 added lines, -7 removed lines

@@ -51,18 +51,18 @@ 
         $options = [
             'bin/dbconsole',
             'sql:execute',
-            '--only-instances=' . $this->instanceName,
+            '--only-instances='.$this->instanceName,
             '--execute-in-process',
-            '--execution-strategy=' . static::EXECUTION_STRATEGY,
+            '--execution-strategy='.static::EXECUTION_STRATEGY,
             '--output-type=json',
         ];
 
         switch ($action) {
             case self::EXECUTE_COMMAND:
-                $options[] = '--sql=' . $sqlOrFilename;
+                $options[] = '--sql='.$sqlOrFilename;
                 break;
             case self::EXECUTE_FILE:
-                $options[] = '--file=' . $sqlOrFilename;
+                $options[] = '--file='.$sqlOrFilename;
                 break;
             default:
                 throw new \OutOfBoundsException('Unsupported action: $action');
@@ -72,13 +72,13 @@ 
 
         if ($instanceConfiguration != $this->databaseConfiguration) {
             if (isset($this->databaseConfiguration['dbname'])) {
-                $options[] = '--database=' . $this->databaseConfiguration['dbname'];
+                $options[] = '--database='.$this->databaseConfiguration['dbname'];
             }
             if (isset($this->databaseConfiguration['user'])) {
-                $options[] = '--user=' . $this->databaseConfiguration['user'];
+                $options[] = '--user='.$this->databaseConfiguration['user'];
             }
             if (isset($this->databaseConfiguration['password'])) {
-                $options[] = '--password=' . $this->databaseConfiguration['password'];
+                $options[] = '--password='.$this->databaseConfiguration['password'];
             }
         }
 

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/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];
             }