phplist

NOTE:: Before reporting an issue, make sure you are running the latest version, currently 3.3.1


View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0011179phplist applicationBatch Processingpublic28-08-07 21:1629-08-07 12:41
Reportertimeskipper 
PrioritynormalSeverityminorReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
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.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0031195)
user1308
29-08-07 12: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)


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker