View Issue Details

IDProjectCategoryView StatusLast Update
0011179phplist applicationBatch Processingpublic29-08-07 12:41
Reportertimeskipper 
PrioritynormalSeverityminorReproducibilityN/A
Status closedResolutionfixed 
Product Version 
Target VersionFixed in Version4.0.x 
Summary0011179: Throttle notification sending when importing users?
DescriptionI 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.
TagsNo tags attached.

Activities

user1308

29-08-07 12:41

  ~0031195

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)