View Issue Details

IDProjectCategoryView StatusLast Update
0004411phplist applicationMessage Managementpublic18-02-08 14:16
Reporterjvarrone 
PrioritynormalSeveritytrivialReproducibilityalways
Status resolvedResolutionfixed 
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.

Relationships

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

Activities

jvarrone

17-10-05 00:37

reporter   ~0007790

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

08-08-06 19:31

manager   ~0016473

fixed in CVS