[buddypress-trac] [BuddyPress Trac] #6271: Separate activity WHERE conditions into new method

buddypress-trac noreply at wordpress.org
Sun Mar 1 20:46:02 UTC 2015


#6271: Separate activity WHERE conditions into new method
-----------------------------------+------------------
 Reporter:  johnjamesjacoby        |       Owner:
     Type:  enhancement            |      Status:  new
 Priority:  low                    |   Milestone:  2.3
Component:  Component - Activity   |     Version:  1.6
 Severity:  minor                  |  Resolution:
 Keywords:  2nd-opinion has-patch  |
-----------------------------------+------------------

Comment (by johnjamesjacoby):

 We could also break up the individual conditions, which would make them
 easier to test.

 Notifications does use this method in two places, but we will likely end
 up eliminating that extra count query to better match what's currently in
 the Activity component.

 The addition of Notification meta means under similar scrutinization, so
 we can apply our conclusion there as well.

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


More information about the buddypress-trac mailing list