View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0015248 | mantisbt | public | 2012-11-29 04:15 | 2014-09-23 18:05 | |
Reporter | Chewits | Assigned To | dregad | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.2.11 | ||||
Target Version | 1.2.13 | Fixed in Version | 1.2.13 | ||
Summary | 0015248: The order of sending emails is inverted when using cron | ||||
Description | When I use cron for sending email notification all the messages seems are sent in the wrong order. I investigated the issue and found that e-mails sent via the function email_send_all() [core/email_api.php], which in its turn uses the email_queue_get_ids() [core/email_queue_api.php]: /**
All the messages are put in the database in a forward order. Unclear why they are retrieved in reverse order? | ||||
Steps To Reproduce | cut 'DESC' from db-query | ||||
Tags | No tags attached. | ||||
MantisBT: master-1.2.x 0737f330 2012-11-29 05:15 Details Diff |
Make email_send_all() send older queued messages first To achieve this, a new optional parameter was added to function email_queue_get_ids() to specify the desired sort order (defaults to 'DESC' to ensure no change in behavior). email_send_all() now retrieves the list of emails to send in ascending order. Fixes 0015248 |
Affected Issues 0015248 |
|
mod - core/email_api.php | Diff File | ||
mod - core/email_queue_api.php | Diff File | ||
MantisBT: master 17858aee 2012-11-29 05:15 Details Diff |
Make email_send_all() send older queued messages first To achieve this, a new optional parameter was added to function email_queue_get_ids() to specify the desired sort order (defaults to 'DESC' to ensure no change in behavior). email_send_all() now retrieves the list of emails to send in ascending order. Fixes 0015248 |
Affected Issues 0015248 |
|
mod - core/email_api.php | Diff File | ||
mod - core/email_queue_api.php | Diff File |