Currently there is no way to disable processing on cron for sites that process the queue on their own. If you set the limits to zero then they will process infinitely. If you set it to 1 it will still process some mails. And, if you happen to hit a lucky coincidence where the one row processed by notifications is also processed by the separate script then that user will receive multiple copies of the e-mail.

I think we need a checkbox for "call notifications via cron" and if it's not set then notifications_cron will immediately return.

This has the added benefit of letting the "queue limitations" still be enforced on the other script in spite of cron not being called the normal way.

CommentFileSizeAuthor
#1 737238_do_not_process_on_cron.patch1.73 KBgreggles

Comments

greggles’s picture

Status: Active » Needs review
StatusFileSize
new1.73 KB
jose reyero’s picture

Status: Needs review » Fixed

Looks good, committed.

It would be nice if you could share the drush command too.

greggles’s picture

Thanks, Jose!

Yes, once it gets to a better state I would like to share it. Right now the drush code is somewhat site specific and not very clean but I will certainly do my best to improve that and share soon.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.