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
0004411phplist applicationMessage Managementpublic17-10-05 00:5118-02-08 14:16
PlatformOSOS Version
Product Version2.10.1 
Target Version2.10.7Fixed in Version2.10.3 
Summary0004411: Requeue message uses original time to calculate how long the message took to send.
DescriptionIf I re-queue a message and send it. The "time to send" incorrectly calculates the time from when the message was first sent. Example, a message was re-queued and was sent very quickly (to only 11 users). However, the display showed: "It took 932 hours 37 mins 17 secs to send this message"

I suggest that when the message is requeued, the message time and date be updated to reflect the current time/date. The subsequent calculation would then be correct.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to 0003721closed phplist 2.10.x 
has duplicate 0004412resolvedmichiel Requeue message uses original time to calculate how long the message took to send. 
related to 0015533resolvedmichiel Wrong processing time reported after processing requeued message 

-  Notes
jvarrone (reporter)
17-10-05 01:37

I corrected the issue with a small modification to the /admin/messages.php file. On or around line 77, make the change indicated below. Adding ", sendstart = now() " to the sql update statement.

  # requeue the message in $resend
  print $GLOBALS['I18N']->get("Requeuing")." $resend ..";
// OLD statement
// $result = Sql_query("update ".$tables["message"]." set status = \"submitted\" where id = $resend");

// NEW Statement. JV:updated message date/time to reflect the rescheduled time
  $result = Sql_query("update ".$tables["message"]." set status = \"submitted\", sendstart = now() where id = $resend");
michiel (manager)
08-08-06 20:31

fixed in CVS

Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker