doctrine / migrations

lib/Doctrine/DBAL/Migrations/Provider/LazySchemaDiffProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
     }
 
     /**
-     * @return Schema
+     * @return \ProxyManager\Proxy\VirtualProxyInterface
      */
     public function createFromSchema()
     {

Spacing +4 added lines, -4 removed lines

@@ -70,7 +70,7 @@ 
 
         return $this->proxyFactory->createProxy(
             Schema::class,
-            function (& $wrappedObject, $proxy, $method, array $parameters, & $initializer) use ($originalSchemaManipulator) {
+            function(& $wrappedObject, $proxy, $method, array $parameters, & $initializer) use ($originalSchemaManipulator) {
                 $initializer   = null;
                 $wrappedObject = $originalSchemaManipulator->createFromSchema();
 
@@ -87,10 +87,10 @@ 
     {
         $originalSchemaManipulator = $this->originalSchemaManipulator;
 
-        if ($fromSchema instanceof LazyLoadingInterface && ! $fromSchema->isProxyInitialized()) {
+        if ($fromSchema instanceof LazyLoadingInterface && !$fromSchema->isProxyInitialized()) {
             return $this->proxyFactory->createProxy(
                 Schema::class,
-                function (& $wrappedObject, $proxy, $method, array $parameters, & $initializer) use ($originalSchemaManipulator, $fromSchema) {
+                function(& $wrappedObject, $proxy, $method, array $parameters, & $initializer) use ($originalSchemaManipulator, $fromSchema) {
                     $initializer   = null;
                     $wrappedObject = $originalSchemaManipulator->createToSchema($fromSchema);
 
@@ -111,7 +111,7 @@ 
     public function getSqlDiffToMigrate(Schema $fromSchema, Schema $toSchema)
     {
         if ($toSchema instanceof LazyLoadingInterface
-            && ! $toSchema->isProxyInitialized()) {
+            && !$toSchema->isProxyInitialized()) {
             return [];
         }
 

lib/Doctrine/DBAL/Migrations/Tools/Console/Helper/ConfigurationHelper.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
          * instead of any other one.
          */
         if ($input->getOption('configuration')) {
-            $outputWriter->write("Loading configuration from command option: " . $input->getOption('configuration'));
+            $outputWriter->write("Loading configuration from command option: ".$input->getOption('configuration'));
 
             return $this->loadConfig($input->getOption('configuration'), $outputWriter);
         }

lib/Doctrine/DBAL/Migrations/Finder/GlobFinder.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     {
         $dir = $this->getRealPath($directory);
 
-        $files = glob(rtrim($dir, '/') . '/Version*.php');
+        $files = glob(rtrim($dir, '/').'/Version*.php');
 
         return $this->loadMigrations($files, $namespace);
     }

lib/Doctrine/DBAL/Migrations/Tools/Console/Command/MigrateCommand.php

Doc Comments +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
      * @param string $versionAlias
      * @param OutputInterface $output
      * @param Configuration $configuration
-     * @return bool|string
+     * @return false|string
      */
     private function getVersionNameFromAlias($versionAlias, OutputInterface $output, Configuration $configuration)
     {

Spacing +5 added lines, -5 removed lines

@@ -104,7 +104,7 @@ 
         }
 
         $executedUnavailableMigrations = array_diff($executedMigrations, $availableMigrations);
-        if ( ! empty($executedUnavailableMigrations)) {
+        if (!empty($executedUnavailableMigrations)) {
             $output->writeln(sprintf(
                 '<error>WARNING! You have %s previously executed migrations'
                 . ' in the database that are not registered migrations.</error>',
@@ -120,7 +120,7 @@ 
             }
 
             $question = 'Are you sure you wish to continue? (y/n)';
-            if ( ! $this->canExecute($question, $input, $output)) {
+            if (!$this->canExecute($question, $input, $output)) {
                 $output->writeln('<error>Migration cancelled!</error>');
 
                 return 1;
@@ -137,12 +137,12 @@ 
 
         $cancelled = false;
         $migration->setNoMigrationException($input->getOption('allow-no-migration'));
-        $result = $migration->migrate($version, $dryRun, $timeAllqueries, function () use ($input, $output, &$cancelled) {
+        $result = $migration->migrate($version, $dryRun, $timeAllqueries, function() use ($input, $output, &$cancelled) {
             $question    = 'WARNING! You are about to execute a database migration'
                 . ' that could result in schema changes and data lost.'
                 . ' Are you sure you wish to continue? (y/n)';
             $canContinue = $this->canExecute($question, $input, $output);
-            $cancelled   = ! $canContinue;
+            $cancelled   = !$canContinue;
 
             return $canContinue;
         });
@@ -172,7 +172,7 @@ 
      */
     private function canExecute($question, InputInterface $input, OutputInterface $output)
     {
-        if ($input->isInteractive() && ! $this->askConfirmation($question, $input, $output)) {
+        if ($input->isInteractive() && !$this->askConfirmation($question, $input, $output)) {
             return false;
         }
 

lib/Doctrine/DBAL/Migrations/FileQueryWriter.php

Doc Comments +3 added lines

@@ -42,6 +42,9 @@
      */
     private $outputWriter;
 
+    /**
+     * @param OutputWriter $outputWriter
+     */
     public function __construct(string $columnName, string $tableName, ?OutputWriter $outputWriter)
     {
         $this->columnName   = $columnName;

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\DBAL\Migrations;
 

lib/Doctrine/DBAL/Migrations/Version.php

Indentation +1 added lines, -1 removed lines

@@ -234,7 +234,7 @@
          * properly, so I had to violate LoD here (so please, let's find a way to solve it on v2).
          */
         return $this->configuration->getQueryWriter()
-                                   ->write($path, $direction, $sqlQueries);
+                                    ->write($path, $direction, $sqlQueries);
     }
 
     /**

Spacing +13 added lines, -13 removed lines

@@ -170,14 +170,14 @@ 
         if (is_array($sql)) {
             foreach ($sql as $key => $query) {
                 $this->sql[] = $query;
-                if ( ! empty($params[$key])) {
+                if (!empty($params[$key])) {
                     $queryTypes = $types[$key] ?? [];
                     $this->addQueryParams($params[$key], $queryTypes);
                 }
             }
         } else {
             $this->sql[] = $sql;
-            if ( ! empty($params)) {
+            if (!empty($params)) {
                 $this->addQueryParams($params, $types);
             }
         }
@@ -207,11 +207,11 @@ 
     {
         $queries = $this->execute($direction, true);
 
-        if ( ! empty($this->params)) {
+        if (!empty($this->params)) {
             throw MigrationException::migrationNotConvertibleToSql($this->class);
         }
 
-        $this->outputWriter->write("\n-- Version " . $this->version . "\n");
+        $this->outputWriter->write("\n-- Version ".$this->version."\n");
 
         $sqlQueries = [$this->version => $queries];
 
@@ -264,12 +264,12 @@ 
             $this->state = self::STATE_PRE;
             $fromSchema  = $this->schemaProvider->createFromSchema();
 
-            $this->migration->{'pre' . ucfirst($direction)}($fromSchema);
+            $this->migration->{'pre'.ucfirst($direction)}($fromSchema);
 
             if ($direction === self::DIRECTION_UP) {
-                $this->outputWriter->write("\n" . sprintf('  <info>++</info> migrating <comment>%s</comment>', $this->version) . "\n");
+                $this->outputWriter->write("\n".sprintf('  <info>++</info> migrating <comment>%s</comment>', $this->version)."\n");
             } else {
-                $this->outputWriter->write("\n" . sprintf('  <info>--</info> reverting <comment>%s</comment>', $this->version) . "\n");
+                $this->outputWriter->write("\n".sprintf('  <info>--</info> reverting <comment>%s</comment>', $this->version)."\n");
             }
 
             $this->state = self::STATE_EXEC;
@@ -282,9 +282,9 @@ 
             $this->executeRegisteredSql($dryRun, $timeAllQueries);
 
             $this->state = self::STATE_POST;
-            $this->migration->{'post' . ucfirst($direction)}($toSchema);
+            $this->migration->{'post'.ucfirst($direction)}($toSchema);
 
-            if ( ! $dryRun) {
+            if (!$dryRun) {
                 if ($direction === self::DIRECTION_UP) {
                     $this->markMigrated();
                 } else {
@@ -392,13 +392,13 @@ 
 
     private function executeRegisteredSql($dryRun = false, $timeAllQueries = false)
     {
-        if ( ! $dryRun) {
-            if ( ! empty($this->sql)) {
+        if (!$dryRun) {
+            if (!empty($this->sql)) {
                 foreach ($this->sql as $key => $query) {
                     $queryStart = microtime(true);
 
-                    if ( ! isset($this->params[$key])) {
-                        $this->outputWriter->write('     <comment>-></comment> ' . $query);
+                    if (!isset($this->params[$key])) {
+                        $this->outputWriter->write('     <comment>-></comment> '.$query);
                         $this->connection->executeQuery($query);
                     } else {
                         $this->outputWriter->write(sprintf('    <comment>-</comment> %s (with parameters)', $query));

lib/Doctrine/DBAL/Migrations/SqlFileWriter.php

Spacing +4 added lines, -4 removed lines

@@ -78,7 +78,7 @@ 
         $string = $this->buildMigrationFile($queriesByVersion, $direction);
 
         if ($this->outputWriter) {
-            $this->outputWriter->write("\n" . sprintf('Writing migration file to "<info>%s</info>"', $path));
+            $this->outputWriter->write("\n".sprintf('Writing migration file to "<info>%s</info>"', $path));
         }
 
         return file_put_contents($path, $string);
@@ -89,10 +89,10 @@ 
         $string = sprintf("-- Doctrine Migration File Generated on %s\n", date('Y-m-d H:i:s'));
 
         foreach ($queriesByVersion as $version => $queries) {
-            $string .= "\n-- Version " . $version . "\n";
+            $string .= "\n-- Version ".$version."\n";
 
             foreach ($queries as $query) {
-                $string .= $query . ";\n";
+                $string .= $query.";\n";
             }
 
 
@@ -119,7 +119,7 @@ 
 
         if (is_dir($path)) {
             $path = realpath($path);
-            $path = $path . '/doctrine_migration_' . date('YmdHis') . '.sql';
+            $path = $path.'/doctrine_migration_'.date('YmdHis').'.sql';
         }
 
         return $path;

lib/Doctrine/DBAL/Migrations/Migration.php

Indentation +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
          * properly, so I had to violate LoD here (so please, let's find a way to solve it on v2).
          */
         return $this->configuration->getQueryWriter()
-                                   ->write($path, $direction, $sql);
+                                    ->write($path, $direction, $sql);
     }
 
     /**

Spacing +4 added lines, -4 removed lines

@@ -140,7 +140,7 @@ 
          * migrations.
          */
         $migrations = $this->configuration->getMigrations();
-        if ( ! isset($migrations[$to]) && $to > 0) {
+        if (!isset($migrations[$to]) && $to > 0) {
             throw MigrationException::unknownMigrationVersion($to);
         }
 
@@ -155,11 +155,11 @@ 
          * means we are already at the destination return an empty array()
          * to signify that there is nothing left to do.
          */
-        if ($from === $to && empty($migrationsToExecute) && ! empty($migrations)) {
+        if ($from === $to && empty($migrationsToExecute) && !empty($migrations)) {
             return $this->noMigrations();
         }
 
-        if ( ! $dryRun && false === $this->migrationsCanExecute($confirm)) {
+        if (!$dryRun && false === $this->migrationsCanExecute($confirm)) {
             return [];
         }
 
@@ -170,7 +170,7 @@ 
         /**
          * If there are no migrations to execute throw an exception.
          */
-        if (empty($migrationsToExecute) && ! $this->noMigrationException) {
+        if (empty($migrationsToExecute) && !$this->noMigrationException) {
             throw MigrationException::noMigrationsToExecute();
         } elseif (empty($migrationsToExecute)) {
             return $this->noMigrations();

lib/Doctrine/DBAL/Migrations/MigrationsVersion.php

Spacing +1 added lines, -1 removed lines

@@ -42,6 +42,6 @@
      */
     private static function isACustomPharBuild($gitversion)
     {
-        return $gitversion !== '@' . 'git-version@';
+        return $gitversion !== '@'.'git-version@';
     }
 }