search

OCA / queue

Asynchronous Job Queue
GNU Affero General Public License v3.0
173 stars 442 forks source link

[FIX] `JobsTrap` `perform_enqueued_jobs` not working with jobs creating new jobs (issue OCA#651) #652

Closed fd-oncodna closed 1 month ago

fd-oncodna commented 2 months ago

JobsTrap perform_enqueued_jobs not working with jobs creating new jobs. After a call to JobsTrap.perform_enqueued_jobs the enqueued_jobs attribute is empty, even if one of the jobs executed had the effect of creating new jobs. That is because after executing the jobs of the queue in a loop, JobsTrap.perform_enqueued_jobs sets the enqueued_jobs to []ignoring the newly created jobs.

OCA-git-bot commented 2 months ago

Hi @guewen, some modules you are maintaining are being modified, check this out!

fd-oncodna commented 1 month ago

@simahawk everything's done ;)

simahawk commented 1 month ago

/ocabot merge patch

OCA-git-bot commented 1 month ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-652-by-simahawk-bump-patch, awaiting test results.

simahawk commented 1 month ago

@fd-oncodna tnx, any chance you can backoprt this to 15 and 16?

OCA-git-bot commented 1 month ago

Congratulations, your PR was merged at 20d343613ea26c6a4eacbff4b5b0aa81c7817b81. Thanks a lot for contributing to OCA. ❤️

fd-oncodna commented 1 month ago

@fd-oncodna tnx, any chance you can backoprt this to 15 and 16?

Yes, see #655 and #656