[buddypress-trac] [BuddyPress Trac] #7205: Adds 'orderby' support to Activity queries

[buddypress-trac]

#7205: Adds 'orderby' support to Activity queries
-------------------------+-----------------------------
 Reporter:  Mamaduka     |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Future Release
Component:  Activity     |     Version:
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |
-------------------------+-----------------------------
Changes (by tw2113):

 * keywords:  needs-patch => has-patch


Comment:

 Definitely possible. Something to keep in mind is that we current order by
 2 items. date_recorded column and then activity ID. We should probably
 leave activity ID as the 2nd order by.

 First up would be providing a parsed value for the method. I feel we
 should have it default to what we have now. Since it's never been
 changeable before, I'm unsure there's much back compat to break.

 Second part, that I think we should do, is only allow values from the
 current columns through. So we'd need to do some sort of comparison or
 array value checks from our allowed columns. Perhaps fall back to
 date_recorded if the user-provided value isn't valid.

 See attachment for consideration.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7205#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac