Closed (won't fix)
Project:
MailQ (Mail Queue)
Version:
5.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Mar 2008 at 13:50 UTC
Updated:
28 Feb 2010 at 09:43 UTC
Once emails have been queued, they are repeatedly sent every time cron is run. This is due to an error in the SQL that selects mail to send. Within mailq_cron()
$rs = db_query_range('SELECT * FROM {mailq_active_queue} WHERE (queue_status != %d OR queue_status != %d)
ORDER BY queue_priority DESC', MAILQ_STATUS_SENT, MAILQ_STATUS_FAILED, 0, variable_get('mailq_batch_size', 10));the OR in the where clause should be changed to AND.
$rs = db_query_range('SELECT * FROM {mailq_active_queue} WHERE (queue_status != %d AND queue_status != %d)
ORDER BY queue_priority DESC', MAILQ_STATUS_SENT, MAILQ_STATUS_FAILED, 0, variable_get('mailq_batch_size', 10));
Comments
Comment #1
zyxware commentedff1 - Thanks for the bug report and fix. This has been fixed in the D6 version of the module. I am marking the issue as wont-fix for D5.