Completed
Push — master ( e46858...d7101c )
by Christophe
10s
created
lib/Doctrine/DBAL/Migrations/Configuration/AbstractFileConfiguration.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -61,13 +61,13 @@  discard block
 block discarded – undo
61 61
 
62 62
     protected function setConfiguration(array $config)
63 63
     {
64
-        foreach($config as $configurationKey => $configurationValue) {
64
+        foreach ($config as $configurationKey => $configurationValue) {
65 65
             if (!isset($this->configurationProperties[$configurationKey])) {
66 66
                 $msg = sprintf('Migrations configuration key "%s" does not exists.', $configurationKey);
67 67
                 throw MigrationException::configurationNotValid($msg);
68 68
             }
69 69
         }
70
-        foreach($this->configurationProperties as $configurationKey => $configurationSetter) {
70
+        foreach ($this->configurationProperties as $configurationKey => $configurationSetter) {
71 71
             if (isset($config[$configurationKey])) {
72 72
                 $this->{$configurationSetter}($config[$configurationKey]);
73 73
             }
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
         } else if (strcasecmp($migrationOrganisation, static::VERSIONS_ORGANIZATION_BY_YEAR_AND_MONTH) == 0) {
97 97
             $this->setMigrationsAreOrganizedByYearAndMonth();
98 98
         } else {
99
-            $msg = 'Unknown ' . var_export($migrationOrganisation, true) . ' for configuration "organize_migrations".';
99
+            $msg = 'Unknown '.var_export($migrationOrganisation, true).' for configuration "organize_migrations".';
100 100
             throw MigrationException::configurationNotValid($msg);
101 101
         }
102 102
     }
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
         if ($this->loaded) {
114 114
             throw MigrationException::configurationFileAlreadyLoaded();
115 115
         }
116
-        if (file_exists($path = getcwd() . '/' . $file)) {
116
+        if (file_exists($path = getcwd().'/'.$file)) {
117 117
             $file = $path;
118 118
         }
119 119
         $this->file = $file;
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 
129 129
     protected function getDirectoryRelativeToFile($file, $input)
130 130
     {
131
-        $path = realpath(dirname($file) . '/' . $input);
131
+        $path = realpath(dirname($file).'/'.$input);
132 132
 
133 133
         return ($path !== false) ? $path : $input;
134 134
     }
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/MigrationsVersion.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,6 +40,6 @@
 block discarded – undo
40 40
      * in a modified (not tagged) phar version.
41 41
      */
42 42
     private static function isACustomPharBuild($gitversion) {
43
-        return $gitversion !== '@' . 'git-version@';
43
+        return $gitversion !== '@'.'git-version@';
44 44
     }
45 45
 }
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Finder/AbstractFinder.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
             $version = (string) substr($className, 7);
63 63
             if ($version === '0') {
64 64
                 throw new \InvalidArgumentException(sprintf(
65
-                    'Cannot load a migrations with the name "%s" because it is a reserved number by doctrine migraitons' . PHP_EOL .
65
+                    'Cannot load a migrations with the name "%s" because it is a reserved number by doctrine migraitons'.PHP_EOL.
66 66
                     'It\'s used to revert all migrations including the first one.',
67 67
                     $version
68 68
                 ));
@@ -78,8 +78,8 @@  discard block
 block discarded – undo
78 78
      *
79 79
      * @return callable
80 80
      */
81
-    protected function getFileSortCallback(){
82
-        return function ($a, $b) {
81
+    protected function getFileSortCallback() {
82
+        return function($a, $b) {
83 83
             return (basename($a) < basename($b)) ? -1 : 1;
84 84
         };
85 85
     }
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Finder/GlobFinder.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
     {
39 39
         $dir = $this->getRealPath($directory);
40 40
 
41
-        $files = glob(rtrim($dir, '/') . '/Version*.php');
41
+        $files = glob(rtrim($dir, '/').'/Version*.php');
42 42
 
43 43
         return $this->loadMigrations($files, $namespace);
44 44
     }
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Version.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
     /** @var SchemaDiffProviderInterface */
103 103
     private $schemaProvider;
104 104
 
105
-    public function __construct(Configuration $configuration, $version, $class, SchemaDiffProviderInterface $schemaProvider=null)
105
+    public function __construct(Configuration $configuration, $version, $class, SchemaDiffProviderInterface $schemaProvider = null)
106 106
     {
107 107
         $this->configuration = $configuration;
108 108
         $this->outputWriter = $configuration->getOutputWriter();
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
         if ($schemaProvider !== null) {
115 115
             $this->schemaProvider = $schemaProvider;
116 116
         }
117
-        if($schemaProvider === null) {
117
+        if ($schemaProvider === null) {
118 118
             $schemaProvider = new SchemaDiffProvider($this->connection->getSchemaManager(),
119 119
                 $this->connection->getDatabasePlatform());
120 120
             $this->schemaProvider = LazySchemaDiffProvider::fromDefaultProxyFacyoryConfiguration($schemaProvider);
@@ -220,11 +220,11 @@  discard block
 block discarded – undo
220 220
     {
221 221
         $queries = $this->execute($direction, true);
222 222
 
223
-        if ( ! empty($this->params)) {
223
+        if (!empty($this->params)) {
224 224
             throw MigrationException::migrationNotConvertibleToSql($this->class);
225 225
         }
226 226
 
227
-        $this->outputWriter->write("\n-- Version " . $this->version . "\n");
227
+        $this->outputWriter->write("\n-- Version ".$this->version."\n");
228 228
 
229 229
         $sqlQueries = [$this->version => $queries];
230 230
         $sqlWriter = new SqlFileWriter(
@@ -276,12 +276,12 @@  discard block
 block discarded – undo
276 276
             $this->state = self::STATE_PRE;
277 277
             $fromSchema = $this->schemaProvider->createFromSchema();
278 278
 
279
-            $this->migration->{'pre' . ucfirst($direction)}($fromSchema);
279
+            $this->migration->{'pre'.ucfirst($direction)}($fromSchema);
280 280
 
281 281
             if ($direction === self::DIRECTION_UP) {
282
-                $this->outputWriter->write("\n" . sprintf('  <info>++</info> migrating <comment>%s</comment>', $this->version) . "\n");
282
+                $this->outputWriter->write("\n".sprintf('  <info>++</info> migrating <comment>%s</comment>', $this->version)."\n");
283 283
             } else {
284
-                $this->outputWriter->write("\n" . sprintf('  <info>--</info> reverting <comment>%s</comment>', $this->version) . "\n");
284
+                $this->outputWriter->write("\n".sprintf('  <info>--</info> reverting <comment>%s</comment>', $this->version)."\n");
285 285
             }
286 286
 
287 287
             $this->state = self::STATE_EXEC;
@@ -294,9 +294,9 @@  discard block
 block discarded – undo
294 294
             $this->executeRegisteredSql($dryRun, $timeAllQueries);
295 295
 
296 296
             $this->state = self::STATE_POST;
297
-            $this->migration->{'post' . ucfirst($direction)}($toSchema);
297
+            $this->migration->{'post'.ucfirst($direction)}($toSchema);
298 298
 
299
-            if (! $dryRun) {
299
+            if (!$dryRun) {
300 300
                 if ($direction === self::DIRECTION_UP) {
301 301
                     $this->markMigrated();
302 302
                 } else {
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
                 }
336 336
             }
337 337
 
338
-            $this->outputWriter->write(sprintf("\n  <info>SS</info> skipped (Reason: %s)",  $e->getMessage()));
338
+            $this->outputWriter->write(sprintf("\n  <info>SS</info> skipped (Reason: %s)", $e->getMessage()));
339 339
 
340 340
             $this->state = self::STATE_NONE;
341 341
 
@@ -398,13 +398,13 @@  discard block
 block discarded – undo
398 398
 
399 399
     private function executeRegisteredSql($dryRun = false, $timeAllQueries = false)
400 400
     {
401
-        if (! $dryRun) {
401
+        if (!$dryRun) {
402 402
             if (!empty($this->sql)) {
403 403
                 foreach ($this->sql as $key => $query) {
404 404
                     $queryStart = microtime(true);
405 405
 
406
-                    if ( ! isset($this->params[$key])) {
407
-                        $this->outputWriter->write('     <comment>-></comment> ' . $query);
406
+                    if (!isset($this->params[$key])) {
407
+                        $this->outputWriter->write('     <comment>-></comment> '.$query);
408 408
                         $this->connection->executeQuery($query);
409 409
                     } else {
410 410
                         $this->outputWriter->write(sprintf('    <comment>-</comment> %s (with parameters)', $query));
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Migration.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -197,7 +197,7 @@
 block discarded – undo
197 197
         return [];
198 198
     }
199 199
 
200
-    private function migrationsCanExecute(callable $confirm=null)
200
+    private function migrationsCanExecute(callable $confirm = null)
201 201
     {
202 202
         return null === $confirm ? true : $confirm();
203 203
     }
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Tools/Console/Command/MigrateCommand.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -177,7 +177,7 @@
 block discarded – undo
177 177
      * @param string $versionAlias
178 178
      * @param OutputInterface $output
179 179
      * @param Configuration $configuration
180
-     * @return bool|string
180
+     * @return false|string
181 181
      */
182 182
     private function getVersionNameFromAlias($versionAlias, OutputInterface $output, Configuration $configuration)
183 183
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
             }
115 115
 
116 116
             $question = 'Are you sure you wish to continue? (y/n)';
117
-            if (! $this->canExecute($question, $input, $output)) {
117
+            if (!$this->canExecute($question, $input, $output)) {
118 118
                 $output->writeln('<error>Migration cancelled!</error>');
119 119
 
120 120
                 return 1;
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
 
132 132
         $cancelled = false;
133 133
         $migration->setNoMigrationException($input->getOption('allow-no-migration'));
134
-        $result = $migration->migrate($version, $dryRun, $timeAllqueries, function () use ($input, $output, &$cancelled) {
134
+        $result = $migration->migrate($version, $dryRun, $timeAllqueries, function() use ($input, $output, &$cancelled) {
135 135
             $question = 'WARNING! You are about to execute a database migration'
136 136
                 . ' that could result in schema changes and data lost.'
137 137
                 . ' Are you sure you wish to continue? (y/n)';
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
      */
167 167
     private function canExecute($question, InputInterface $input, OutputInterface $output)
168 168
     {
169
-        if ($input->isInteractive() && ! $this->askConfirmation($question, $input, $output)) {
169
+        if ($input->isInteractive() && !$this->askConfirmation($question, $input, $output)) {
170 170
             return false;
171 171
         }
172 172
 
Please login to merge, or discard this patch.
lib/Doctrine/DBAL/Migrations/Tools/Console/Command/DiffCommand.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
      */
44 44
     protected $schemaProvider;
45 45
 
46
-    public function __construct(SchemaProviderInterface $schemaProvider=null)
46
+    public function __construct(SchemaProviderInterface $schemaProvider = null)
47 47
     {
48 48
         $this->schemaProvider = $schemaProvider;
49 49
         parent::__construct();
@@ -93,10 +93,10 @@  discard block
 block discarded – undo
93 93
         $toSchema = $this->getSchemaProvider()->createSchema();
94 94
 
95 95
         //Not using value from options, because filters can be set from config.yml
96
-        if ( ! $isDbalOld && $filterExpr = $conn->getConfiguration()->getFilterSchemaAssetsExpression()) {
96
+        if (!$isDbalOld && $filterExpr = $conn->getConfiguration()->getFilterSchemaAssetsExpression()) {
97 97
             foreach ($toSchema->getTables() as $table) {
98 98
                 $tableName = $table->getName();
99
-                if ( ! preg_match($filterExpr, $this->resolveTableName($tableName))) {
99
+                if (!preg_match($filterExpr, $this->resolveTableName($tableName))) {
100 100
                     $toSchema->dropTable($tableName);
101 101
                 }
102 102
             }
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
             $input->getOption('line-length')
116 116
         );
117 117
 
118
-        if (! $up && ! $down) {
118
+        if (!$up && !$down) {
119 119
             $output->writeln('No changes detected in your mapping information.');
120 120
 
121 121
             return;
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
         $output->writeln(file_get_contents($path));
129 129
     }
130 130
 
131
-    private function buildCodeFromSql(Configuration $configuration, array $sql, $formatted=false, $lineLength=120)
131
+    private function buildCodeFromSql(Configuration $configuration, array $sql, $formatted = false, $lineLength = 120)
132 132
     {
133 133
         $currentPlatform = $configuration->getConnection()->getDatabasePlatform()->getName();
134 134
         $code = [];
Please login to merge, or discard this patch.
Configuration/Connection/Loader/ConnectionConfigurationChainLoader.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
      */
42 42
     public function chosen()
43 43
     {
44
-        foreach($this->loaders as $loader) {
44
+        foreach ($this->loaders as $loader) {
45 45
             if (null !== $confObj = $loader->chosen()) {
46 46
                 return $confObj;
47 47
             }
Please login to merge, or discard this patch.