doctrine / migrations

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

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

Spacing +7 added lines, -7 removed lines

@@ -97,8 +97,8 @@ 
         $replacements = [
             $configuration->getMigrationsNamespace(),
             $version,
-            $up ? '        ' . implode("\n        ", explode("\n", $up)) : null,
-            $down ? '        ' . implode("\n        ", explode("\n", $down)) : null,
+            $up ? '        '.implode("\n        ", explode("\n", $up)) : null,
+            $down ? '        '.implode("\n        ", explode("\n", $down)) : null,
         ];
 
         $code = str_replace($placeHolders, $replacements, $this->getTemplate());
@@ -106,14 +106,14 @@ 
 
         $directoryHelper = new MigrationDirectoryHelper($configuration);
         $dir             = $directoryHelper->getMigrationDirectory();
-        $path            = $dir . '/Version' . $version . '.php';
+        $path            = $dir.'/Version'.$version.'.php';
 
         file_put_contents($path, $code);
 
         $editorCmd = $input->getOption('editor-cmd');
 
         if ($editorCmd) {
-            proc_open($editorCmd . ' ' . escapeshellarg($path), [], $pipes);
+            proc_open($editorCmd.' '.escapeshellarg($path), [], $pipes);
         }
 
         return $path;
@@ -127,16 +127,16 @@ 
             return;
         }
 
-        if (! is_file($customTemplate) || ! is_readable($customTemplate)) {
+        if (!is_file($customTemplate) || !is_readable($customTemplate)) {
             throw new \InvalidArgumentException(
-                'The specified template "' . $customTemplate . '" cannot be found or is not readable.'
+                'The specified template "'.$customTemplate.'" cannot be found or is not readable.'
             );
         }
 
         $content = file_get_contents($customTemplate);
 
         if ($content === false) {
-            throw new \InvalidArgumentException('The specified template "' . $customTemplate . '" could not be read.');
+            throw new \InvalidArgumentException('The specified template "'.$customTemplate.'" could not be read.');
         }
 
         $output->writeln(sprintf('Using custom migration template "<info>%s</info>"', $customTemplate));

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

Spacing +6 added lines, -6 removed lines

@@ -70,7 +70,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.');
         }
 
@@ -126,26 +126,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)) {
-            if (! $all) {
+            if (!$all) {
                 throw new \InvalidArgumentException(sprintf('The version "%s" already exists in the version table.', $version));
             }
             $marked = true;
         }
 
-        if (! $this->markMigrated && ! $this->configuration->hasVersionMigrated($version)) {
-            if (! $all) {
+        if (!$this->markMigrated && !$this->configuration->hasVersionMigrated($version)) {
+            if (!$all) {
                 throw new \InvalidArgumentException(sprintf('The version "%s" does not exist in the version table.', $version));
             }
             $marked = false;
         }
 
-        if (! isset($marked)) {
+        if (!isset($marked)) {
             if ($this->markMigrated) {
                 $version->markMigrated();
             } else {

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

Spacing +8 added lines, -8 removed lines

@@ -56,10 +56,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('');
     }
 
@@ -79,7 +79,7 @@ 
      */
     protected function getMigrationConfiguration(InputInterface $input, OutputInterface $output)
     {
-        if (! $this->migrationConfiguration) {
+        if (!$this->migrationConfiguration) {
             if ($this->getHelperSet()->has('configuration')
                 && $this->getHelperSet()->get('configuration') instanceof ConfigurationHelperInterface) {
                 $configHelper = $this->getHelperSet()->get('configuration');
@@ -106,8 +106,8 @@ 
      */
     private function getOutputWriter(OutputInterface $output)
     {
-        if (! $this->outputWriter) {
-            $this->outputWriter = new OutputWriter(function ($message) use ($output) {
+        if (!$this->outputWriter) {
+            $this->outputWriter = new OutputWriter(function($message) use ($output) {
                 return $output->writeln($message);
             });
         }
@@ -133,7 +133,7 @@ 
                 new ConnectionConfigurationLoader($this->configuration),
             ]
         );
-        $connection  = $chainLoader->chosen();
+        $connection = $chainLoader->chosen();
 
         if ($connection) {
             return $this->connection = $connection;

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

Spacing +1 added lines, -1 removed lines

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