Merge pull request #107 from usemarkup/feat/user-manage-jobs-non-prod

feat(recurring): all jobs can be disabled when not on prod

Author: calumbrodie

Resources/config/services.yml

@@ -24,6 +24,7 @@ services:
         arguments:
             - '%kernel.root_dir%'
             - '@Markup\JobQueueBundle\Repository\JobStatusRepository'
+            - '%kernel.environment%'
     markup_job_queue.writer.supervisord_config_file:
         class: Markup\JobQueueBundle\Service\SupervisordConfigFileWriter
         arguments:

Service/RecurringConsoleCommandReader.php

@@ -41,12 +41,19 @@ class RecurringConsoleCommandReader
     /** @var JobStatusRepository */
     private $jobStatusRepository;
 
+    /**
+     * @var string
+     */
+    private $kernelEnv;
+
     public function __construct(
         string $kernelPath,
-        JobStatusRepository $jobStatusRepository
+        JobStatusRepository $jobStatusRepository,
+        string $kernelEnv
     ) {
         $this->kernelPath = $kernelPath;
         $this->jobStatusRepository = $jobStatusRepository;
+        $this->kernelEnv = $kernelEnv;
     }
 
     public function setConfigurationFileName($name)
@@ -140,6 +147,11 @@ private function parseConfiguration(array $config)
                 throw new InvalidConfigurationException(sprintf('`arguments` config key must be an array for %s', $group['command']));
             }
 
+            // user management by default when environment is not prod
+            if ($this->kernelEnv !== 'prod' && !isset($group['user_managed'])) {
+                $group['user_managed'] = true;
+            }
+
             $recurringConsoleCommandConfiguration = new RecurringConsoleCommandConfiguration(
                 $group['command'],
                 $group['arguments'] ?? [],