largersmallernormaltext version of this page

Viewing Issue Simple Details Jump to Notes ] View Advanced ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0005521 [phplist] Message Send Process minor always 20-02-06 12:29 18-02-08 14:19
Reporter michiel View Status public  
Assigned To
Priority normal Resolution open  
Status new   Product Version
Summary 0005521: check on "doneusers" needs to use "status" column
Description The check on "users who have received this message" checks for an entry in the usermessage table, but doesn't take the status into account, so it will also exclude users who haven't actually received the message.

Additional Information
Tags No tags attached.
Attached Files

- Relationships
related to 0005522assignedmichiel done users may become too large to fit in memory 
related to 0002705closed PHPList v2.11 release 

-  Notes
(0010857)
michiel (administrator)
20-02-06 12:31

change query to

  $req = Sql_Query("select userid from {$tables["usermessage"]} where messageid = $messageid and status = \"sent\"");
(0010858)
michiel (administrator)
20-02-06 12:42

Hmm, it may not be as simple as that. When using RSS, and there is no content for the user to send, the status becomes "not sent" and if this is the case for at least one user in the total time of "frequency" (eg a day), the message would never be marked sent. So either RSS messages need a different code to be indentified, or it needs to stay like this.

Situation:

Using RSS message, user "last sent" data is after the time of sending the message. Therefore at sending a message time, there is no content for the user to send. The message will remain in the queue as being not done yet, but if the message is marked "sent" for the user instead (which is not true, but would help), the message is marked "sent" in total the next time the queue is run, as there will not be any more users, and therefore requeued for the next "frequency" time.


Mantis 1.1.6[^]
Copyright © 2000 - 2008 Mantis Group
Powered by Mantis Bugtracker