View Issue Details

IDProjectCategoryView StatusLast Update
0005521phpList 3 applicationMessage Send Processpublic21-06-18 13:05
Reportermichiel 
PrioritynormalSeverityminorReproducibilityalways
Status newResolutionopen 
Product Version 
Target Versionnext majorFixed in Version 
Summary0005521: check on "doneusers" needs to use "status" column
DescriptionThe 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.

TagsNo tags attached.

Relationships

related to 0005522 resolvedmichiel phpList 3 application done users may become too large to fit in memory 
related to 0002705 closed phpList 3 application PHPList v2.11 release 
related to 0016788 new phpList 3 plugins Not able to send a message to subscribers who previously were excluded 

Activities

michiel

20-02-06 12:31

manager   ~0010857

change query to

  $req = Sql_Query("select userid from {$tables["usermessage"]} where messageid = $messageid and status = \"sent\"");

michiel

20-02-06 12:42

manager   ~0010858

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.