pkp#9682 Decoupled the execution of scheduled tasks to the execution …

…of jobs through a configuration
ipula · Feb 22, 2024 · 3c35376 · 3c35376
1 parent 6b39372
commit 3c35376
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/classes/task/ProcessQueueJobs.php b/classes/task/ProcessQueueJobs.php
@@ -37,7 +37,7 @@ public function getName()
      */
     public function executeActions()
     {
-        if (Application::isUnderMaintenance()) {
+        if (Application::isUnderMaintenance() || !Config::getVar('queues', 'job_runner', true)) {
             return true;
         }
 
@@ -49,7 +49,7 @@ public function executeActions()
             return true;
         }
 
-        // Run queue jobs on CLI
+        // Executes all pending jobs when running the runScheduledTasks.php on the CLI
         if (runOnCLI('runScheduledTasks.php')) {
             while ($jobBuilder->count()) {
                 $jobQueue->runJobInQueue();
@@ -58,15 +58,13 @@ public function executeActions()
             return true;
         }
 
-        // Run queue jobs off CLI
-        if (Config::getVar('queues', 'job_runner', false)) {
-            (new JobRunner($jobQueue))
-                ->withMaxExecutionTimeConstrain()
-                ->withMaxJobsConstrain()
-                ->withMaxMemoryConstrain()
-                ->withEstimatedTimeToProcessNextJobConstrain()
-                ->processJobs($jobBuilder);
-        }
+        // Executes a limited number of jobs when processing a request
+        (new JobRunner($jobQueue))
+            ->withMaxExecutionTimeConstrain()
+            ->withMaxJobsConstrain()
+            ->withMaxMemoryConstrain()
+            ->withEstimatedTimeToProcessNextJobConstrain()
+            ->processJobs($jobBuilder);
 
         return true;
     }