In RC4 of PrivateMSG new messages are listed in the wrong order (by Date/Last Update).

Have a look at the screenshot to see, what I mean.

CommentFileSizeAuthor
#1 privatemsg_order_fix.patch957 bytesberdir
privatemsg_newmsg_order.png36.43 KBgold-dupe

Comments

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
StatusFileSize
new privatemsg_order_fix.patch957 bytes

Currently, the list is sorted by is_new, which is an alias for SUM(is_new). That does not make sense, instead, it should use the old MAX(is_new) because we don't care how many new messages a thread has.

naheemsays’s picture

naheemsays commented

yes, this makes sense - untested as my setup is still... evolving, but I think this should go in.

(Saying that, I like the semantic effect that using sum has... potentially more important unread messages will be higher up...)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Version: 6.x-1.0-rc4 » 7.x-1.x-dev
Status: Needs review » Fixed

Fixed in 6.x-1.x-dev and 7.x-1.x-dev.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.