zikula-modules / InterCom

Messenger for Zikula
16 stars 7 forks source link

Archive feature how it supposed to work? #71

Closed Kaik closed 7 years ago

Kaik commented 9 years ago

Hi, archive feature in db is represented by stored property which is boolean. This indicates that one message can be stored or not but it not tell us who stored the message. So either the archive feature works: only for inbox and for message sender and outbox messages cannot be stored or stored message by sender user displays as stored for recipient which is very bad or stored message is actually a copy of a message with only one user set - this is bad as well as stored message is not having the users data (sender/recipient whichever stored it)

My solution is to make this field 2 digit (1,0) figure so if sender archives the message then it will be 1x if recipient x1 both 11 none 00 then searching for sender will be "where stored = 10 or stored = 11"

cmfcmf commented 9 years ago

You could also consider to split the field into two fields to be more clear, i.e. StoredBySender and StoredByRecipient

Kaik commented 7 years ago

done