doctrine / migrations

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

Spacing +2 added lines, -2 removed lines

@@ -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/Tools/Console/Command/DiffCommand.php

Spacing +4 added lines, -4 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();
@@ -90,10 +90,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);
                 }
             }
@@ -102,7 +102,7 @@ 
         $up = $this->buildCodeFromSql($configuration, $fromSchema->getMigrateToSql($toSchema, $platform));
         $down = $this->buildCodeFromSql($configuration, $fromSchema->getMigrateFromSql($toSchema, $platform));
 
-        if (! $up && ! $down) {
+        if (!$up && !$down) {
             $output->writeln('No changes detected in your mapping information.', 'ERROR');
 
             return;

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

Spacing +6 added lines, -6 removed lines

@@ -92,7 +92,7 @@ 
     {
         $this->configuration = $this->getMigrationConfiguration($input, $output);
 
-        if ( ! $input->getOption('add') && ! $input->getOption('delete')) {
+        if (!$input->getOption('add') && !$input->getOption('delete')) {
             throw new \InvalidArgumentException('You must specify whether you want to --add or --delete the specified version.');
         }
 
@@ -147,26 +147,26 @@ 
 
     private function mark($version, $all = false)
     {
-        if ( ! $this->configuration->hasVersion($version)) {
+        if (!$this->configuration->hasVersion($version)) {
             throw MigrationException::unknownMigrationVersion($version);
         }
 
         $version = $this->configuration->getVersion($version);
         if ($this->markMigrated && $this->configuration->hasVersionMigrated($version)) {
             $marked = true;
-            if (! $all) {
+            if (!$all) {
                 throw new \InvalidArgumentException(sprintf('The version "%s" already exists in the version table.', $version));
             }
         }
 
-        if ( ! $this->markMigrated && ! $this->configuration->hasVersionMigrated($version)) {
+        if (!$this->markMigrated && !$this->configuration->hasVersionMigrated($version)) {
             $marked = false;
-            if (! $all) {
+            if (!$all) {
                 throw new \InvalidArgumentException(sprintf('The version "%s" does not exists in the version table.', $version));
             }
         }
 
-        if ( ! isset($marked)) {
+        if (!isset($marked)) {
             if ($this->markMigrated) {
                 $version->markMigrated();
             } else {

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

Spacing +4 added lines, -4 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;
@@ -128,11 +128,11 @@ 
             $dryRun = (boolean) $input->getOption('dry-run');
 
             // warn the user if no dry run and interaction is on
-            if (! $dryRun) {
+            if (!$dryRun) {
                 $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)';
-                if (! $this->canExecute($question, $input, $output)) {
+                if (!$this->canExecute($question, $input, $output)) {
                     $output->writeln('<error>Migration cancelled!</error>');
 
                     return 1;
@@ -156,7 +156,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/Tools/Console/Command/StatusCommand.php

Spacing +4 added lines, -4 removed lines

@@ -96,7 +96,7 @@ 
             'Next Version'                      => $formattedVersions['next'],
             'Latest Version'                    => $formattedVersions['latest'],
             'Executed Migrations'               => count($executedMigrations),
-            'Executed Unavailable Migrations'   => $numExecutedUnavailableMigrations > 0 ? '<error>'.$numExecutedUnavailableMigrations.'</error>' : 0,
+            'Executed Unavailable Migrations'   => $numExecutedUnavailableMigrations > 0 ? '<error>' . $numExecutedUnavailableMigrations . '</error>' : 0,
             'Available Migrations'              => count($availableMigrations),
             'New Migrations'                    => $newMigrations > 0 ? '<question>' . $newMigrations . '</question>' : 0
         ];
@@ -123,7 +123,7 @@ 
     {
         $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 = '';
@@ -134,8 +134,8 @@ 
 
             $output->writeln('    <comment>>></comment> ' . $formattedVersion .
                 ' (<comment>' . $version->getVersion() . '</comment>)' .
-                str_repeat(' ', 49 - strlen($formattedVersion) - strlen($version->getVersion()))  .
-                $status  . $migrationDescription);
+                str_repeat(' ', 49 - strlen($formattedVersion) - strlen($version->getVersion())) .
+                $status . $migrationDescription);
         }
     }
 }

Doc Comments +3 added lines

@@ -122,6 +122,9 @@
         }
     }
 
+    /**
+     * @param \Doctrine\DBAL\Migrations\Version[] $migrations
+     */
     private function showVersions($migrations, Configuration $configuration, OutputInterface $output)
     {
         $migratedVersions = $configuration->getMigratedVersions();

lib/Doctrine/DBAL/Migrations/Configuration/AbstractFileConfiguration.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
 
     protected function setConfiguration(Array $config)
     {
-        foreach($config as $configurationKey => $configurationValue) {
+        foreach ($config as $configurationKey => $configurationValue) {
             if (!isset($this->configurationProperties[$configurationKey])) {
                 $msg = sprintf('Migrations configuration key "%s" does not exists.', $configurationKey);
                 throw MigrationException::configurationNotValid($msg);

lib/Doctrine/DBAL/Migrations/Configuration/Configuration.php

Spacing +1 added lines, -1 removed lines

@@ -787,7 +787,7 @@
      */
     private function ensureMigrationClassExists($class)
     {
-        if ( ! class_exists($class)) {
+        if (!class_exists($class)) {
             throw MigrationException::migrationClassNotFound($class, $this->getMigrationsNamespace());
         }
     }

lib/Doctrine/DBAL/Migrations/Version.php

Spacing +4 added lines, -4 removed lines

@@ -270,7 +270,7 @@ 
             $this->migration->{'post' . ucfirst($direction)}($toSchema);
             $this->executeRegisteredSql($dryRun, $timeAllQueries);
 
-            if (! $dryRun) {
+            if (!$dryRun) {
                 if ($direction === self::DIRECTION_UP) {
                     $this->markMigrated();
                 } else {
@@ -309,7 +309,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;
 
@@ -372,12 +372,12 @@ 
 
     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])) {
+                    if (!isset($this->params[$key])) {
                         $this->outputWriter->write('     <comment>-></comment> ' . $query);
                         $this->connection->exec($query);
                     } else {

Doc Comments +9 added lines, -2 removed lines

@@ -105,6 +105,10 @@ 
      */
     private $state = self::STATE_NONE;
 
+    /**
+     * @param string $version
+     * @param string $class
+     */
     public function __construct(Configuration $configuration, $version, $class)
     {
         $this->configuration = $configuration;
@@ -198,7 +202,7 @@ 
      * @param string $path      The path to write the migration SQL file.
      * @param string $direction The direction to execute.
      *
-     * @return boolean $written
+     * @return integer $written
      */
     public function writeSqlFile($path, $direction = self::DIRECTION_UP)
     {
@@ -348,6 +352,9 @@ 
         }
     }
 
+    /**
+     * @param double $queryStart
+     */
     private function outputQueryTime($queryStart, $timeAllQueries = false)
     {
         if ($timeAllQueries !== false) {
@@ -361,7 +368,7 @@ 
     /**
      * Returns the time this migration version took to execute
      *
-     * @return integer $time The time this migration version took to execute
+     * @return double $time The time this migration version took to execute
      */
     public function getTime()
     {