use proper table sorting (http://api.drupal.org/api/search/6/tablesort) inside privatemsg_privatemsg_list_sent_alter and privatemsg_privatemsg_list_alter instead of current ordering method.

Comments

naheemsays’s picture

naheemsays commented

I don't think we can use table sort API if there is another sort already in the query.

Before the patch by mrtoner, I had in another issue also removed the tablesort api as when combining it with a sort from the query, I was getting an error over an incorrect sql statement (the query ended up like "ORDER BY new desc ORDER BY mid" or similar)

So, tips would be welcome on how to fix that. (My personal favourite approach - get rid of any choice. The users get what we give them.)

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Closed (duplicate)

With my patch at #348907: Per thread/Multiple thread actions, the lists do now use tablesort_sql.