View GitHub Repository DoctrineMigrationsBundle

Schedule Inspection
Subscribe
Completed
Pull Request — master (#294)
by Asmir
11:15
created
Status
Category
Toggle Patches
DependencyInjection/Configuration.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch 
@@ -51,13 +51,13 @@  discard block
 
 block discarded – undo
51 51 
                     ->defaultValue([])
52 52 
                     ->prototype('scalar')->end()
53 53 
                     ->validate()
54 
-                        ->ifTrue(static function ($v) {
54 
+                        ->ifTrue(static function($v) {
55 55 
                             return count($v) === 0;
56 56 
                         })
57 57 
                         ->thenInvalid('At least one migrations path must be specified.')
58 58
59 
-                        ->ifTrue(static function ($v) {
60 
-                            return count($v) >  1;
59 
+                        ->ifTrue(static function($v) {
60 
+                            return count($v) > 1;
61 61 
                         })
62 62 
                         ->thenInvalid('Maximum one migration path can be specified with the 2.x version.')
63 63 
                     ->end()
 
@@ -74,8 +74,8 @@  discard block
 
 block discarded – undo
74 74 
                                 ->scalarNode('version_column_length')
75 75 
                                     ->defaultValue(null)
76 76 
                                     ->validate()
77 
-                                        ->ifTrue(static function ($v) {
78 
-                                            return $v< 1024;
77 
+                                        ->ifTrue(static function($v) {
78 
+                                            return $v < 1024;
79 79 
                                         })
80 80 
                                         ->thenInvalid('The minimum length for the version column is 1024.')
81 81 
                                     ->end()
 
@@ -115,7 +115,7 @@  discard block
 
 block discarded – undo
115 115 
                 ->scalarNode('organize_migrations')->defaultValue(false)
116 116 
                     ->info('Organize migrations mode. Possible values are: "BY_YEAR", "BY_YEAR_AND_MONTH", false')
117 117 
                     ->validate()
118 
-                        ->ifTrue(static function ($v) use ($organizeMigrationModes) {
118 
+                        ->ifTrue(static function($v) use ($organizeMigrationModes) {
119 119 
                             if ($v === false) {
120 120 
                                 return false;
121 121 
                             }
 
@@ -130,8 +130,8 @@  discard block
 
 block discarded – undo
130 130 
                     ->end()
131 131 
                     ->validate()
132 132 
                         ->ifString()
133 
-                            ->then(static function ($v) {
134 
-                                return constant('Doctrine\Migrations\Configuration\Configuration::VERSIONS_ORGANIZATION_' . strtoupper($v));
133 
+                            ->then(static function($v) {
134 
+                                return constant('Doctrine\Migrations\Configuration\Configuration::VERSIONS_ORGANIZATION_'.strtoupper($v));
135 135 
                             })
136 136 
                     ->end()
137 137 
                 ->end()
Please login to merge, or discard this patch.
DependencyInjection/DoctrineMigrationsExtension.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -29,7 +29,7 @@  discard block
 
 block discarded – undo
29 29 
         $config = $this->processConfiguration($configuration, $configs);
30 30
31 31 
         // 3.x forward compatibility layer
32 
-        if (isset($config['migrations_paths']) && count($config['migrations_paths'])>0) {
32 
+        if (isset($config['migrations_paths']) && count($config['migrations_paths']) > 0) {
33 33 
             $config['namespace'] = key($config['migrations_paths']);
34 34 
             $config['dir_name']  = current($config['migrations_paths']);
35 35 
             unset($config['migrations_paths']);
 
@@ -53,10 +53,10 @@  discard block
 
 block discarded – undo
53 53 
         }
54 54
55 55 
         foreach ($config as $key => $value) {
56 
-            $container->setParameter($this->getAlias() . '.' . $key, $value);
56 
+            $container->setParameter($this->getAlias().'.'.$key, $value);
57 57 
         }
58 58
59 
-        $locator = new FileLocator(__DIR__ . '/../Resources/config/');
59 
+        $locator = new FileLocator(__DIR__.'/../Resources/config/');
60 60 
         $loader  = new XmlFileLoader($container, $locator);
61 61
62 62 
         $loader->load('services.xml');
 
@@ -69,7 +69,7 @@  discard block
 
 block discarded – undo
69 69 
      */
70 70 
     public function getXsdValidationBasePath() : string
71 71 
     {
72 
-        return __DIR__ . '/../Resources/config/schema';
72 
+        return __DIR__.'/../Resources/config/schema';
73 73 
     }
74 74
75 75 
     public function getNamespace() : string
Please login to merge, or discard this patch.