Merge pull request #84 from digiaonline/fix-apply-migration-options

Fix bug with parsing command options in ApplyMigrationCommand
digiaonline · Mar 15, 2018 · 8fd4e79 · 8fd4e79
2 parents 9e9b3a2 + 822b816
commit 8fd4e79
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/Console/ApplyMigrationCommand.php b/src/Console/ApplyMigrationCommand.php
@@ -25,7 +25,7 @@ class ApplyMigrationCommand extends AbstractCommand
      */
     protected $signature = 'elastic:migrations:migrate 
                             { config : The path to the index configuration file } 
-                            { --batchSize= : The number of documents to handle per batch while re-indexing }
+                            { --batchSize=' . self::DEFAULT_BATCH_SIZE . ' : The number of documents to handle per batch while re-indexing }
                             { --updateAllTypes : Forces update across all types }';
 
     /**
@@ -41,8 +41,8 @@ class ApplyMigrationCommand extends AbstractCommand
     public function handle()
     {
         $configurationPath = (string)$this->argument('config');
-        $batchSize         = (int)$this->option('batchSize', self::DEFAULT_BATCH_SIZE);
-        $updateAllTypes    = (bool)$this->option('updateAllTypes', false);
+        $batchSize         = (int)$this->option('batchSize');
+        $updateAllTypes    = (bool)$this->option('updateAllTypes');
 
         $pipeline = new Pipeline([
             new DetermineTargetVersionStage(),