doctrine / migrations

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

Doc Comments +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 
     /**
      * Transform the recursiveIterator result array of array into the expected array of migration file
-     * @param iterable|string[][] $iteratorFilesMatch
+     * @param \RegexIterator $iteratorFilesMatch
      * @return string[]
      */
     private function getMatches($iteratorFilesMatch)

lib/Doctrine/DBAL/Migrations/OutputWriter.php

Doc Comments +3 added lines

@@ -12,6 +12,9 @@
     /** @var \Closure|null */
     private $closure;
 
+    /**
+     * @param \Closure $closure
+     */
     public function __construct(?\Closure $closure = null)
     {
         if ($closure === null) {

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function __construct(?\Closure $closure = null)
     {
         if ($closure === null) {
-            $closure = function ($message) {
+            $closure = function($message) {
             };
         }
         $this->closure = $closure;

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

Doc Comments +1 added lines, -1 removed lines

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

Spacing +5 added lines, -5 removed lines

@@ -85,7 +85,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>',
@@ -101,7 +101,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;
@@ -118,12 +118,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;
         });
@@ -151,7 +151,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

Doc Comments +6 added lines

@@ -69,11 +69,17 @@
         }
     }
 
+    /**
+     * @param string $title
+     */
     private function writeStatusInfosLineAligned(OutputInterface $output, $title, $value)
     {
         $output->writeln('    <comment>>></comment> ' . $title . ': ' . str_repeat(' ', 50 - strlen($title)) . $value);
     }
 
+    /**
+     * @param \Doctrine\DBAL\Migrations\Version[] $migrations
+     */
     private function showVersions($migrations, Configuration $configuration, OutputInterface $output)
     {
         $migratedVersions = $configuration->getMigratedVersions();

Spacing +10 added lines, -10 removed lines

@@ -42,10 +42,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;
+                $value = $value > 0 ? '<error>'.$value.'</error>' : 0;
             }
             $this->writeStatusInfosLineAligned($output, $name, $value);
         }
@@ -62,8 +62,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>)');
                 }
             }
         }
@@ -71,7 +71,7 @@ 
 
     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)
@@ -83,15 +83,15 @@ 
             $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/Tools/Console/Helper/ConfigurationHelper.php

Doc Comments +7 added lines

@@ -27,6 +27,10 @@ 
      */
     private $configuration;
 
+    /**
+     * @param Connection $connection
+     * @param Configuration $configuration
+     */
     public function __construct(?Connection $connection = null, ?Configuration $configuration = null)
     {
         $this->connection    = $connection;
@@ -76,6 +80,9 @@ 
     }
 
 
+    /**
+     * @param string $config
+     */
     private function configExists($config)
     {
         return file_exists($config);

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,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);
         }
@@ -66,7 +66,7 @@ 
         ];
         foreach ($defaultConfig as $config) {
             if ($this->configExists($config)) {
-                $outputWriter->write('Loading configuration from file: ' . $config);
+                $outputWriter->write('Loading configuration from file: '.$config);
 
                 return $this->loadConfig($config, $outputWriter);
             }

lib/Doctrine/DBAL/Migrations/Version.php

Doc Comments +15 added lines, -1 removed lines

@@ -81,6 +81,10 @@ 
     /** @var SchemaDiffProviderInterface */
     private $schemaProvider;
 
+    /**
+     * @param string $version
+     * @param string $class
+     */
     public function __construct(Configuration $configuration, $version, $class, ?SchemaDiffProviderInterface $schemaProvider = null)
     {
         $this->configuration = $configuration;
@@ -137,6 +141,9 @@ 
         $this->markVersion('up');
     }
 
+    /**
+     * @param string $direction
+     */
     private function markVersion($direction)
     {
         $action = $direction === 'up' ? 'insert' : 'delete';
@@ -157,7 +164,7 @@ 
      * Add some SQL queries to this versions migration
      *
      * @param string|string[]     $sql
-     * @param mixed[]|mixed[][]   $params
+     * @param string[]   $params
      * @param string[]|string[][] $types
      */
     public function addSql($sql, array $params = [], array $types = [])
@@ -360,6 +367,9 @@ 
         }
     }
 
+    /**
+     * @param double $queryStart
+     */
     private function outputQueryTime($queryStart, $timeAllQueries = false)
     {
         if ($timeAllQueries !== false) {
@@ -464,6 +474,10 @@ 
         return sprintf('with parameters (%s)', implode(', ', $out));
     }
 
+    /**
+     * @param string $direction
+     * @param boolean $dryRun
+     */
     private function dispatchEvent($eventName, $direction, $dryRun)
     {
         $this->configuration->dispatchEvent($eventName, new MigrationsVersionEventArgs(

Spacing +13 added lines, -13 removed lines

@@ -165,14 +165,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);
             }
         }
@@ -202,11 +202,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];
 
@@ -259,12 +259,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;
@@ -277,9 +277,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 {
@@ -387,13 +387,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/Migration.php

Spacing +4 added lines, -4 removed lines

@@ -118,7 +118,7 @@ 
          * migrations.
          */
         $migrations = $this->configuration->getMigrations();
-        if (! isset($migrations[$to]) && $to > 0) {
+        if (!isset($migrations[$to]) && $to > 0) {
             throw MigrationException::unknownMigrationVersion($to);
         }
 
@@ -133,11 +133,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 && $this->migrationsCanExecute($confirm) === false) {
+        if (!$dryRun && $this->migrationsCanExecute($confirm) === false) {
             return [];
         }
 
@@ -148,7 +148,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/Event/Listeners/AutoCommitListener.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function onMigrationsMigrated(MigrationsEventArgs $args)
     {
         $conn = $args->getConnection();
-        if (! $args->isDryRun() && ! $conn->isAutoCommit()) {
+        if (!$args->isDryRun() && !$conn->isAutoCommit()) {
             $conn->commit();
         }
     }

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

Spacing +8 added lines, -8 removed lines

@@ -75,16 +75,16 @@ 
         $filterExpr = $conn->getConfiguration()->getFilterSchemaAssetsExpression();
 
         //Not using value from options, because filters can be set from config.yml
-        if (! $isDbalOld && $filterExpr) {
+        if (!$isDbalOld && $filterExpr) {
             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);
                 }
             }
         }
 
-        $up   = $this->buildCodeFromSql(
+        $up = $this->buildCodeFromSql(
             $configuration,
             $fromSchema->getMigrateToSql($toSchema, $platform),
             $input->getOption('formatted'),
@@ -97,7 +97,7 @@ 
             $input->getOption('line-length')
         );
 
-        if (! $up && ! $down) {
+        if (!$up && !$down) {
             $output->writeln('No changes detected in your mapping information.');
 
             return;
@@ -123,9 +123,9 @@ 
             }
 
             if ($formatted) {
-                if (! class_exists('\SqlFormatter')) {
+                if (!class_exists('\SqlFormatter')) {
                     throw new \InvalidArgumentException(
-                        'The "--formatted" option can only be used if the sql formatter is installed.' .
+                        'The "--formatted" option can only be used if the sql formatter is installed.'.
                         'Please run "composer require jdorn/sql-formatter".'
                     );
                 }
@@ -140,7 +140,7 @@ 
             $code[] = sprintf('$this->addSql(%s);', var_export($query, true));
         }
 
-        if (! empty($code)) {
+        if (!empty($code)) {
             array_unshift(
                 $code,
                 sprintf(
@@ -157,7 +157,7 @@ 
 
     private function getSchemaProvider()
     {
-        if (! $this->schemaProvider) {
+        if (!$this->schemaProvider) {
             $this->schemaProvider = new OrmSchemaProvider($this->getHelper('entityManager')->getEntityManager());
         }