doctrine / migrations

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 +3 added lines, -3 removed lines

@@ -114,7 +114,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;
@@ -131,7 +131,7 @@ 
 
         $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)';
@@ -166,7 +166,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/Configuration/Configuration.php

Spacing +8 added lines, -8 removed lines

@@ -501,7 +501,7 @@ 
         $this->createMigrationTable();
 
         $version = $this->connection->fetchColumn(
-            "SELECT " . $this->migrationsColumnName . " FROM " . $this->migrationsTableName . " WHERE " . $this->migrationsColumnName . " = ?",
+            "SELECT ".$this->migrationsColumnName." FROM ".$this->migrationsTableName." WHERE ".$this->migrationsColumnName." = ?",
             [$version->getVersion()]
         );
 
@@ -518,7 +518,7 @@ 
         $this->connect();
         $this->createMigrationTable();
 
-        $ret = $this->connection->fetchAll("SELECT " . $this->migrationsColumnName . " FROM " . $this->migrationsTableName);
+        $ret = $this->connection->fetchAll("SELECT ".$this->migrationsColumnName." FROM ".$this->migrationsTableName);
 
         return array_map('current', $ret);
     }
@@ -563,7 +563,7 @@ 
             foreach ($this->migrations as $migration) {
                 $migratedVersions[] = sprintf("'%s'", $migration->getVersion());
             }
-            $where = " WHERE " . $this->migrationsColumnName . " IN (" . implode(', ', $migratedVersions) . ")";
+            $where = " WHERE ".$this->migrationsColumnName." IN (".implode(', ', $migratedVersions).")";
         }
 
         $sql = sprintf("SELECT %s FROM %s%s ORDER BY %s DESC",
@@ -613,7 +613,7 @@ 
 
         $versions = array_map('strval', array_keys($this->migrations));
         array_unshift($versions, '0');
-        $offset = array_search((string)$version, $versions);
+        $offset = array_search((string) $version, $versions);
         if ($offset === false || !isset($versions[$offset + $delta])) {
             // Unknown version or delta out of bounds.
             return null;
@@ -670,7 +670,7 @@ 
         $this->connect();
         $this->createMigrationTable();
 
-        $result = $this->connection->fetchColumn("SELECT COUNT(" . $this->migrationsColumnName . ") FROM " . $this->migrationsTableName);
+        $result = $this->connection->fetchColumn("SELECT COUNT(".$this->migrationsColumnName.") FROM ".$this->migrationsTableName);
 
         return $result !== false ? $result : 0;
     }
@@ -781,7 +781,7 @@ 
      * @param string $eventName The event to emit.
      * @param $args The event args instance to emit.
      */
-    public function dispatchEvent($eventName, EventArgs $args=null)
+    public function dispatchEvent($eventName, EventArgs $args = null)
     {
         $this->connection->getEventManager()->dispatchEvent($eventName, $args);
     }
@@ -826,7 +826,7 @@ 
      * @param DateTimeInterface|null $now Defaults to the current time in UTC
      * @return string The newly generated version
      */
-    public function generateVersionNumber(\DateTimeInterface $now=null)
+    public function generateVersionNumber(\DateTimeInterface $now = null)
     {
         $now = $now ?: new \DateTime('now', new \DateTimeZone('UTC'));
 
@@ -899,7 +899,7 @@ 
      */
     private function ensureMigrationClassExists($class)
     {
-        if ( ! class_exists($class)) {
+        if (!class_exists($class)) {
             throw MigrationException::migrationClassNotFound($class, $this->getMigrationsNamespace());
         }
     }

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

Spacing +5 added lines, -5 removed lines

@@ -43,7 +43,7 @@ 
      */
     protected $schemaProvider;
 
-    public function __construct(SchemaProviderInterface $schemaProvider=null)
+    public function __construct(SchemaProviderInterface $schemaProvider = null)
     {
         $this->schemaProvider = $schemaProvider;
         parent::__construct();
@@ -93,10 +93,10 @@ 
         $toSchema = $this->getSchemaProvider()->createSchema();
 
         //Not using value from options, because filters can be set from config.yml
-        if ( ! $isDbalOld && $filterExpr = $conn->getConfiguration()->getFilterSchemaAssetsExpression()) {
+        if (!$isDbalOld && $filterExpr = $conn->getConfiguration()->getFilterSchemaAssetsExpression()) {
             foreach ($toSchema->getTables() as $table) {
                 $tableName = $table->getName();
-                if ( ! preg_match($filterExpr, $this->resolveTableName($tableName))) {
+                if (!preg_match($filterExpr, $this->resolveTableName($tableName))) {
                     $toSchema->dropTable($tableName);
                 }
             }
@@ -115,7 +115,7 @@ 
             $input->getOption('line-length')
         );
 
-        if (! $up && ! $down) {
+        if (!$up && !$down) {
             $output->writeln('No changes detected in your mapping information.');
 
             return;
@@ -128,7 +128,7 @@ 
         $output->writeln(file_get_contents($path));
     }
 
-    private function buildCodeFromSql(Configuration $configuration, array $sql, $formatted=false, $lineLength=120)
+    private function buildCodeFromSql(Configuration $configuration, array $sql, $formatted = false, $lineLength = 120)
     {
         $currentPlatform = $configuration->getConnection()->getDatabasePlatform()->getName();
         $code = [];

lib/Doctrine/DBAL/Migrations/Configuration/Connection/Loader/ConnectionConfigurationChainLoader.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public function chosen()
     {
-        foreach($this->loaders as $loader) {
+        foreach ($this->loaders as $loader) {
             if (null !== $confObj = $loader->chosen()) {
                 return $confObj;
             }

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

Spacing +12 added lines, -12 removed lines

@@ -64,10 +64,10 @@ 
         $output->writeln("\n <info>==</info> Configuration\n");
         foreach ($infos->getMigrationsInfos() as $name => $value) {
             if ($name == 'New Migrations') {
-                $value = $value > 0 ? '<question>' . $value . '</question>' : 0;
+                $value = $value > 0 ? '<question>'.$value.'</question>' : 0;
             }
-            if($name == 'Executed Unavailable Migrations') {
-                $value = $value > 0 ? '<error>' . $value . '</error>' : 0;
+            if ($name == 'Executed Unavailable Migrations') {
+                $value = $value > 0 ? '<error>'.$value.'</error>' : 0;
             }
             $this->writeStatusInfosLineAligned($output, $name, $value);
         }
@@ -82,8 +82,8 @@ 
             if (count($infos->getExecutedUnavailableMigrations())) {
                 $output->writeln("\n <info>==</info> Previously Executed Unavailable Migration Versions\n");
                 foreach ($infos->getExecutedUnavailableMigrations() as $executedUnavailableMigration) {
-                    $output->writeln('    <comment>>></comment> ' . $configuration->getDateTime($executedUnavailableMigration) .
-                        ' (<comment>' . $executedUnavailableMigration . '</comment>)');
+                    $output->writeln('    <comment>>></comment> '.$configuration->getDateTime($executedUnavailableMigration).
+                        ' (<comment>'.$executedUnavailableMigration.'</comment>)');
                 }
             }
         }
@@ -91,27 +91,27 @@ 
 
     private function writeStatusInfosLineAligned(OutputInterface $output, $title, $value)
     {
-        $output->writeln('    <comment>>></comment> ' . $title . ': ' . str_repeat(' ', 50 - strlen($title)) . $value);
+        $output->writeln('    <comment>>></comment> '.$title.': '.str_repeat(' ', 50 - strlen($title)).$value);
     }
 
     private function showVersions($migrations, Configuration $configuration, OutputInterface $output)
     {
         $migratedVersions = $configuration->getMigratedVersions();
 
-        foreach($migrations as $version) {
+        foreach ($migrations as $version) {
             $isMigrated = in_array($version->getVersion(), $migratedVersions);
             $status = $isMigrated ? '<info>migrated</info>' : '<error>not migrated</error>';
 
             $migrationDescription = $version->getMigration()->getDescription()
-                ? str_repeat(' ', 5) . $version->getMigration()->getDescription()
+                ? str_repeat(' ', 5).$version->getMigration()->getDescription()
                 : '';
 
             $formattedVersion = $configuration->getDateTime($version->getVersion());
 
-            $output->writeln('    <comment>>></comment> ' . $formattedVersion .
-                ' (<comment>' . $version->getVersion() . '</comment>)' .
-                str_repeat(' ', max(1, 49 - strlen($formattedVersion) - strlen($version->getVersion())))  .
-                $status  . $migrationDescription);
+            $output->writeln('    <comment>>></comment> '.$formattedVersion.
+                ' (<comment>'.$version->getVersion().'</comment>)'.
+                str_repeat(' ', max(1, 49 - strlen($formattedVersion) - strlen($version->getVersion()))).
+                $status.$migrationDescription);
         }
     }
 }

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

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
             $version = (string) substr($className, 7);
             if ($version === '0') {
                 throw new \InvalidArgumentException(sprintf(
-                    'Cannot load a migrations with the name "%s" because it is a reserved number by doctrine migrations' . PHP_EOL .
+                    'Cannot load a migrations with the name "%s" because it is a reserved number by doctrine migrations'.PHP_EOL.
                     'It\'s used to revert all migrations including the first one.',
                     $version
                 ));
@@ -78,8 +78,8 @@ 
      *
      * @return callable
      */
-    protected function getFileSortCallback(){
-        return function ($a, $b) {
+    protected function getFileSortCallback() {
+        return function($a, $b) {
             return (basename($a) < basename($b)) ? -1 : 1;
         };
     }

lib/Doctrine/DBAL/Migrations/Version.php

Spacing +13 added lines, -13 removed lines

@@ -103,7 +103,7 @@ 
     /** @var SchemaDiffProviderInterface */
     private $schemaProvider;
 
-    public function __construct(Configuration $configuration, $version, $class, SchemaDiffProviderInterface $schemaProvider=null)
+    public function __construct(Configuration $configuration, $version, $class, SchemaDiffProviderInterface $schemaProvider = null)
     {
         $this->configuration = $configuration;
         $this->outputWriter = $configuration->getOutputWriter();
@@ -115,7 +115,7 @@ 
         if ($schemaProvider !== null) {
             $this->schemaProvider = $schemaProvider;
         }
-        if($schemaProvider === null) {
+        if ($schemaProvider === null) {
             $schemaProvider = new SchemaDiffProvider($this->connection->getSchemaManager(),
                 $this->connection->getDatabasePlatform());
             $this->schemaProvider = LazySchemaDiffProvider::fromDefaultProxyFactoryConfiguration($schemaProvider);
@@ -221,11 +221,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];
 
@@ -284,12 +284,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;
@@ -302,9 +302,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 {
@@ -345,7 +345,7 @@ 
                 }
             }
 
-            $this->outputWriter->write(sprintf("\n  <info>SS</info> skipped (Reason: %s)",  $e->getMessage()));
+            $this->outputWriter->write(sprintf("\n  <info>SS</info> skipped (Reason: %s)", $e->getMessage()));
 
             $this->state = self::STATE_NONE;
 
@@ -411,13 +411,13 @@ 
 
     private function executeRegisteredSql($dryRun = false, $timeAllQueries = false)
     {
-        if (! $dryRun) {
+        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));

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

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

Spacing +4 added lines, -4 removed lines

@@ -77,10 +77,10 @@
     {
         $name = $configuration->getName();
         $name = $name ? $name : 'Doctrine Database Migrations';
-        $name = str_repeat(' ', 20) . $name . str_repeat(' ', 20);
-        $output->writeln('<question>' . str_repeat(' ', strlen($name)) . '</question>');
-        $output->writeln('<question>' . $name . '</question>');
-        $output->writeln('<question>' . str_repeat(' ', strlen($name)) . '</question>');
+        $name = str_repeat(' ', 20).$name.str_repeat(' ', 20);
+        $output->writeln('<question>'.str_repeat(' ', strlen($name)).'</question>');
+        $output->writeln('<question>'.$name.'</question>');
+        $output->writeln('<question>'.str_repeat(' ', strlen($name)).'</question>');
         $output->writeln('');
     }