Passed
Pull Request — master (#585)
by Grégoire
04:53
created
DBAL/Migrations/Tools/Console/Helper/MigrationStatusInfosHelper.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
         }
75 75
 
76 76
         //Show normal version number
77
-        return $this->configuration->getDateTime($version) . ' (<comment>' . $version . '</comment>)';
77
+        return $this->configuration->getDateTime($version).' (<comment>'.$version.'</comment>)';
78 78
     }
79 79
 
80 80
     /**
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/QueryWriter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\DBAL\Migrations;
6 6
 
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Tools/Console/Command/AbstractCommand.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -61,10 +61,10 @@  discard block
 block discarded – undo
61 61
     {
62 62
         $name = $configuration->getName();
63 63
         $name = $name ? $name : 'Doctrine Database Migrations';
64
-        $name = str_repeat(' ', 20) . $name . str_repeat(' ', 20);
65
-        $output->writeln('<question>' . str_repeat(' ', strlen($name)) . '</question>');
66
-        $output->writeln('<question>' . $name . '</question>');
67
-        $output->writeln('<question>' . str_repeat(' ', strlen($name)) . '</question>');
64
+        $name = str_repeat(' ', 20).$name.str_repeat(' ', 20);
65
+        $output->writeln('<question>'.str_repeat(' ', strlen($name)).'</question>');
66
+        $output->writeln('<question>'.$name.'</question>');
67
+        $output->writeln('<question>'.str_repeat(' ', strlen($name)).'</question>');
68 68
         $output->writeln('');
69 69
     }
70 70
 
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
      */
88 88
     protected function getMigrationConfiguration(InputInterface $input, OutputInterface $output)
89 89
     {
90
-        if ( ! $this->migrationConfiguration) {
90
+        if (!$this->migrationConfiguration) {
91 91
             if ($this->getHelperSet()->has('configuration')
92 92
                 && $this->getHelperSet()->get('configuration') instanceof ConfigurationHelperInterface) {
93 93
                 $configHelper = $this->getHelperSet()->get('configuration');
@@ -118,8 +118,8 @@  discard block
 block discarded – undo
118 118
      */
119 119
     private function getOutputWriter(OutputInterface $output)
120 120
     {
121
-        if ( ! $this->outputWriter) {
122
-            $this->outputWriter = new OutputWriter(function ($message) use ($output) {
121
+        if (!$this->outputWriter) {
122
+            $this->outputWriter = new OutputWriter(function($message) use ($output) {
123 123
                 return $output->writeln($message);
124 124
             });
125 125
         }
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
                 new ConnectionConfigurationLoader($this->configuration),
148 148
             ]
149 149
         );
150
-        $connection  = $chainLoader->chosen();
150
+        $connection = $chainLoader->chosen();
151 151
 
152 152
         if ($connection) {
153 153
             return $this->connection = $connection;
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Tools/Console/Command/GenerateCommand.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -100,8 +100,8 @@  discard block
 block discarded – undo
100 100
         $replacements = [
101 101
             $configuration->getMigrationsNamespace(),
102 102
             $version,
103
-            $up ? "        " . implode("\n        ", explode("\n", $up)) : null,
104
-            $down ? "        " . implode("\n        ", explode("\n", $down)) : null,
103
+            $up ? "        ".implode("\n        ", explode("\n", $up)) : null,
104
+            $down ? "        ".implode("\n        ", explode("\n", $down)) : null,
105 105
         ];
106 106
 
107 107
         $code = str_replace($placeHolders, $replacements, $this->getTemplate());
@@ -109,12 +109,12 @@  discard block
 block discarded – undo
109 109
 
110 110
         $directoryHelper = new MigrationDirectoryHelper($configuration);
111 111
         $dir             = $directoryHelper->getMigrationDirectory();
112
-        $path            = $dir . '/Version' . $version . '.php';
112
+        $path            = $dir.'/Version'.$version.'.php';
113 113
 
114 114
         file_put_contents($path, $code);
115 115
 
116 116
         if ($editorCmd = $input->getOption('editor-cmd')) {
117
-            proc_open($editorCmd . ' ' . escapeshellarg($path), [], $pipes);
117
+            proc_open($editorCmd.' '.escapeshellarg($path), [], $pipes);
118 118
         }
119 119
 
120 120
         return $path;
@@ -128,16 +128,16 @@  discard block
 block discarded – undo
128 128
             return;
129 129
         }
130 130
 
131
-        if ( ! is_file($customTemplate) || ! is_readable($customTemplate)) {
131
+        if (!is_file($customTemplate) || !is_readable($customTemplate)) {
132 132
             throw new \InvalidArgumentException(
133
-                'The specified template "' . $customTemplate . '" cannot be found or is not readable.'
133
+                'The specified template "'.$customTemplate.'" cannot be found or is not readable.'
134 134
             );
135 135
         }
136 136
 
137 137
         $content = file_get_contents($customTemplate);
138 138
 
139 139
         if ($content === false) {
140
-            throw new \InvalidArgumentException('The specified template "' . $customTemplate . '" could not be read.');
140
+            throw new \InvalidArgumentException('The specified template "'.$customTemplate.'" could not be read.');
141 141
         }
142 142
 
143 143
         $output->writeln(sprintf('Using custom migration template "<info>%s</info>"', $customTemplate));
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Finder/RecursiveRegexFinder.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
 
46 46
     /**
47 47
      * Transform the recursiveIterator result array of array into the expected array of migration file
48
-     * @param iterable $iteratorFilesMatch
48
+     * @param \RegexIterator $iteratorFilesMatch
49 49
      * @return array
50 50
      */
51 51
     private function getMatches($iteratorFilesMatch)
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Configuration/AbstractFileConfiguration.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
     protected function setConfiguration(array $config)
47 47
     {
48 48
         foreach ($config as $configurationKey => $configurationValue) {
49
-            if ( ! isset($this->configurationProperties[$configurationKey])) {
49
+            if (!isset($this->configurationProperties[$configurationKey])) {
50 50
                 $msg = sprintf('Migrations configuration key "%s" does not exist.', $configurationKey);
51 51
                 throw MigrationException::configurationNotValid($msg);
52 52
             }
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         } elseif (strcasecmp($migrationOrganisation, static::VERSIONS_ORGANIZATION_BY_YEAR_AND_MONTH) == 0) {
81 81
             $this->setMigrationsAreOrganizedByYearAndMonth();
82 82
         } else {
83
-            $msg = 'Unknown ' . var_export($migrationOrganisation, true) . ' for configuration "organize_migrations".';
83
+            $msg = 'Unknown '.var_export($migrationOrganisation, true).' for configuration "organize_migrations".';
84 84
             throw MigrationException::configurationNotValid($msg);
85 85
         }
86 86
     }
@@ -97,12 +97,12 @@  discard block
 block discarded – undo
97 97
         if ($this->loaded) {
98 98
             throw MigrationException::configurationFileAlreadyLoaded();
99 99
         }
100
-        if (file_exists($path = getcwd() . '/' . $file)) {
100
+        if (file_exists($path = getcwd().'/'.$file)) {
101 101
             $file = $path;
102 102
         }
103 103
         $this->file = $file;
104 104
 
105
-        if ( ! file_exists($file)) {
105
+        if (!file_exists($file)) {
106 106
             throw new \InvalidArgumentException('Given config file does not exist');
107 107
         }
108 108
 
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
 
113 113
     protected function getDirectoryRelativeToFile($file, $input)
114 114
     {
115
-        $path = realpath(dirname($file) . '/' . $input);
115
+        $path = realpath(dirname($file).'/'.$input);
116 116
 
117 117
         return ($path !== false) ? $path : $input;
118 118
     }
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Tools/Console/Command/VersionCommand.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
     {
76 76
         $this->configuration = $this->getMigrationConfiguration($input, $output);
77 77
 
78
-        if ( ! $input->getOption('add') && ! $input->getOption('delete')) {
78
+        if (!$input->getOption('add') && !$input->getOption('delete')) {
79 79
             throw new \InvalidArgumentException('You must specify whether you want to --add or --delete the specified version.');
80 80
         }
81 81
 
@@ -131,26 +131,26 @@  discard block
 block discarded – undo
131 131
 
132 132
     private function mark($version, $all = false)
133 133
     {
134
-        if ( ! $this->configuration->hasVersion($version)) {
134
+        if (!$this->configuration->hasVersion($version)) {
135 135
             throw MigrationException::unknownMigrationVersion($version);
136 136
         }
137 137
 
138 138
         $version = $this->configuration->getVersion($version);
139 139
         if ($this->markMigrated && $this->configuration->hasVersionMigrated($version)) {
140
-            if ( ! $all) {
140
+            if (!$all) {
141 141
                 throw new \InvalidArgumentException(sprintf('The version "%s" already exists in the version table.', $version));
142 142
             }
143 143
             $marked = true;
144 144
         }
145 145
 
146
-        if ( ! $this->markMigrated && ! $this->configuration->hasVersionMigrated($version)) {
147
-            if ( ! $all) {
146
+        if (!$this->markMigrated && !$this->configuration->hasVersionMigrated($version)) {
147
+            if (!$all) {
148 148
                 throw new \InvalidArgumentException(sprintf('The version "%s" does not exist in the version table.', $version));
149 149
             }
150 150
             $marked = false;
151 151
         }
152 152
 
153
-        if ( ! isset($marked)) {
153
+        if (!isset($marked)) {
154 154
             if ($this->markMigrated) {
155 155
                 $version->markMigrated();
156 156
             } else {
Please login to merge, or discard this patch.