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

View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0011179phplistBatch Processingpublic28-08-07 20:1629-08-07 11:41
Reportertimeskipper 
PrioritynormalSeverityminorReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version2.11.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

-  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)

Copyright © 2000 - 2010 MantisBT Group
Powered by Mantis Bugtracker