Completed
Pull Request — master (#426)
by Reen
09:08
created
lib/Doctrine/DBAL/Migrations/Tools/Console/Command/VersionCommand.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
     {
93 93
         $this->configuration = $this->getMigrationConfiguration($input, $output);
94 94
 
95
-        if ( ! $input->getOption('add') && ! $input->getOption('delete')) {
95
+        if (!$input->getOption('add') && !$input->getOption('delete')) {
96 96
             throw new \InvalidArgumentException('You must specify whether you want to --add or --delete the specified version.');
97 97
         }
98 98
 
@@ -149,26 +149,26 @@  discard block
 block discarded – undo
149 149
 
150 150
     private function mark($version, $all = false)
151 151
     {
152
-        if ( ! $this->configuration->hasVersion($version)) {
152
+        if (!$this->configuration->hasVersion($version)) {
153 153
             throw MigrationException::unknownMigrationVersion($version);
154 154
         }
155 155
 
156 156
         $version = $this->configuration->getVersion($version);
157 157
         if ($this->markMigrated && $this->configuration->hasVersionMigrated($version)) {
158
-            if (! $all) {
158
+            if (!$all) {
159 159
                 throw new \InvalidArgumentException(sprintf('The version "%s" already exists in the version table.', $version));
160 160
             }
161 161
             $marked = true;
162 162
         }
163 163
 
164
-        if ( ! $this->markMigrated && ! $this->configuration->hasVersionMigrated($version)) {
165
-            if (! $all) {
164
+        if (!$this->markMigrated && !$this->configuration->hasVersionMigrated($version)) {
165
+            if (!$all) {
166 166
                 throw new \InvalidArgumentException(sprintf('The version "%s" does not exists in the version table.', $version));
167 167
             }
168 168
             $marked = false;
169 169
         }
170 170
 
171
-        if ( ! isset($marked)) {
171
+        if (!isset($marked)) {
172 172
             if ($this->markMigrated) {
173 173
                 $version->markMigrated();
174 174
             } else {
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Version.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
     /** @var SchemaDiffProviderInterface */
103 103
     private $schemaProvider;
104 104
 
105
-    public function __construct(Configuration $configuration, $version, $class, SchemaDiffProviderInterface $schemaProvider=null)
105
+    public function __construct(Configuration $configuration, $version, $class, SchemaDiffProviderInterface $schemaProvider = null)
106 106
     {
107 107
         $this->configuration = $configuration;
108 108
         $this->outputWriter = $configuration->getOutputWriter();
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
         if ($schemaProvider !== null) {
115 115
             $this->schemaProvider = $schemaProvider;
116 116
         }
117
-        if($schemaProvider === null) {
117
+        if ($schemaProvider === null) {
118 118
             $schemaProvider = new SchemaDiffProvider($this->connection->getSchemaManager(),
119 119
                 $this->connection->getDatabasePlatform());
120 120
             $this->schemaProvider = LazySchemaDiffProvider::fromDefaultProxyFacyoryConfiguration($schemaProvider);
@@ -296,7 +296,7 @@  discard block
 block discarded – undo
296 296
             $this->state = self::STATE_POST;
297 297
             $this->migration->{'post' . ucfirst($direction)}($toSchema);
298 298
 
299
-            if (! $dryRun) {
299
+            if (!$dryRun) {
300 300
                 if ($direction === self::DIRECTION_UP) {
301 301
                     $this->markMigrated();
302 302
                 } else {
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
                 }
336 336
             }
337 337
 
338
-            $this->outputWriter->write(sprintf("\n  <info>SS</info> skipped (Reason: %s)",  $e->getMessage()));
338
+            $this->outputWriter->write(sprintf("\n  <info>SS</info> skipped (Reason: %s)", $e->getMessage()));
339 339
 
340 340
             $this->state = self::STATE_NONE;
341 341
 
@@ -398,12 +398,12 @@  discard block
 block discarded – undo
398 398
 
399 399
     private function executeRegisteredSql($dryRun = false, $timeAllQueries = false)
400 400
     {
401
-        if (! $dryRun) {
401
+        if (!$dryRun) {
402 402
             if (!empty($this->sql)) {
403 403
                 foreach ($this->sql as $key => $query) {
404 404
                     $queryStart = microtime(true);
405 405
 
406
-                    if ( ! isset($this->params[$key])) {
406
+                    if (!isset($this->params[$key])) {
407 407
                         $this->outputWriter->write('     <comment>-></comment> ' . $query);
408 408
                         $this->connection->executeQuery($query);
409 409
                     } else {
Please login to merge, or discard this patch.