diff options
author | Vincent Petry <vincent@nextcloud.com> | 2022-04-25 16:16:28 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-25 16:16:28 +0300 |
commit | 8f34bc833c859763717c30ce0dc29170585956ed (patch) | |
tree | d14dada29a48a7503cd9da78026e7ec0e17d03f3 | |
parent | d1e244b85928acc71bd482e3c20c756cc3457c95 (diff) | |
parent | edfba7f151a2ef60b73343a29ddbe04df32c0baa (diff) |
Merge pull request #32116 from nextcloud/backport/32109/stable24
[stable24] Get not only time-sensitive next job from list but any when not in cron-mode
-rw-r--r-- | cron.php | 2 | ||||
-rw-r--r-- | lib/private/BackgroundJob/JobList.php | 2 | ||||
-rw-r--r-- | tests/lib/BackgroundJob/DummyJobList.php | 2 |
3 files changed, 4 insertions, 2 deletions
@@ -147,6 +147,7 @@ try { break; } + $logger->debug('CLI cron call has selected job with ID ' . strval($job->getId()), ['app' => 'cron']); $job->execute($jobList, $logger); // clean up after unclean jobs \OC_Util::tearDownFS(); @@ -169,6 +170,7 @@ try { $jobList = \OC::$server->getJobList(); $job = $jobList->getNext(); if ($job != null) { + $logger->debug('WebCron call has selected job with ID ' . strval($job->getId()), ['app' => 'cron']); $job->execute($jobList, $logger); $jobList->setLastJob($job); } diff --git a/lib/private/BackgroundJob/JobList.php b/lib/private/BackgroundJob/JobList.php index 21af79c4686..fe65a1879bc 100644 --- a/lib/private/BackgroundJob/JobList.php +++ b/lib/private/BackgroundJob/JobList.php @@ -203,7 +203,7 @@ class JobList implements IJobList { * @param bool $onlyTimeSensitive * @return IJob|null */ - public function getNext(bool $onlyTimeSensitive = true): ?IJob { + public function getNext(bool $onlyTimeSensitive = false): ?IJob { $query = $this->connection->getQueryBuilder(); $query->select('*') ->from('jobs') diff --git a/tests/lib/BackgroundJob/DummyJobList.php b/tests/lib/BackgroundJob/DummyJobList.php index ec06203a477..0751409f62c 100644 --- a/tests/lib/BackgroundJob/DummyJobList.php +++ b/tests/lib/BackgroundJob/DummyJobList.php @@ -78,7 +78,7 @@ class DummyJobList extends \OC\BackgroundJob\JobList { * @param bool $onlyTimeSensitive * @return IJob|null */ - public function getNext(bool $onlyTimeSensitive = true): ?IJob { + public function getNext(bool $onlyTimeSensitive = false): ?IJob { if (count($this->jobs) > 0) { if ($this->last < (count($this->jobs) - 1)) { $i = $this->last + 1; |