Merge pull request #56 from mothership-gmbh/develop

feature disable-logs more strict
mothership-gmbh · Mar 23, 2017 · 0677ebd · 0677ebd
2 parents 36eda90 + 2c141f8
commit 0677ebd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/lib/Mothership/Magerun/Base/Command/Workflow/AbstractWorkflow.php b/src/lib/Mothership/Magerun/Base/Command/Workflow/AbstractWorkflow.php
@@ -88,7 +88,7 @@ public function run($args = [], $logIntoMagentoScheduler = true)
         $this->magentoRootDir = array_key_exists('root-dir', $args) ? $args['root-dir'] : null;
         $this->yaml = array_key_exists('yaml', $args) ? $args['yaml'] : [];
 
-        if (array_key_exists('disable-logs', $this->input->getOptions()) || (array_key_exists('disable-logs', $this->yaml['class']))) {
+        if ($this->input->getOption('disable-logs') || (array_key_exists('disable-logs', $this->yaml['class']))) {
             $this->areLogsDisabled = true;
         }
 

diff --git a/src/lib/Mothership/Magerun/Base/Command/Workflow/RunCommand.php b/src/lib/Mothership/Magerun/Base/Command/Workflow/RunCommand.php
@@ -80,7 +80,7 @@ protected function configure()
                 'disable-logs',
                 'ds',
                 InputOption::VALUE_OPTIONAL,
-                'disable explicity logs, if not set, logs are automatically enabled'
+                'disable explicity logs, if not set and set to true, logs are automatically enabled'
             );
 
             if (array_key_exists(1, $explodedConfig) && strpos($explodedConfig[1], 'yaml') !== false) {