largersmallernormaltext version of this page
Main | My View | View Issues | Change Log | Roadmap | Docs
  

Viewing Issue Simple Details [ Jump to Notes ] [ View Advanced ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0011179 [phplist] Batch Processing minor N/A 28-08-07 20:16 29-08-07 11:41
Reporter timeskipper View Status public  
Assigned To
Priority normal Resolution fixed  
Status closed   Product Version
Summary 0011179: Throttle notification sending when importing users?
Description I don't know how to reopen a solved ticket so this is for ticket #0010817.

I don't have cvs to check so this is just a question.
In the ticket 0010817, I suggested this:
In the mean time I did this:
- edit admin/import1.php
- find the line:
sendMail($email, getConfig("subscribesubject"), $subscribemessage,system_messageheaders(),$envelope);

- add this right after:
if (MAILQUEUE_THROTTLE) {
sleep(MAILQUEUE_THROTTLE);
}

But I discover an issue with this suggestion:
Doing this will add the sleep delay even if the use already exist in the DB.

It is better to change those:
if (!TEST && $notify == "yes" && $addition) {
          sendMail($email, getConfig("subscribesubject"), $subscribemessage,system_messageheaders(),$envelope);

into:
if (!TEST && $notify == "yes" && $addition) {
          sendMail($email, getConfig("subscribesubject"), $subscribemessage,system_messageheaders(),$envelope);
          if (MAILQUEUE_THROTTLE) {
             sleep(MAILQUEUE_THROTTLE);
          }
        }

So that you do the sleep only if we are sending something.
Additional Information
Tags No tags attached.
Attached Files

- Relationships

-  Notes
(0031195)
hernan (developer)
29-08-07 11:41

 Well, actually that was done, but in a diff way. I have added an input in the inport process to choose wheter to use the throttle or not, and then if new and if can send mail do throttle (default 0)

Mantis 1.1.6[^]
Copyright © 2000 - 2008 Mantis Group
Powered by Mantis Bugtracker