[buddypress-trac] [BuddyPress Trac] #3961: Hierarchical groups

[buddypress-trac]

#3961: Hierarchical groups
------------------------------------+-----------------------
 Reporter:  sooskriszta             |       Owner:  dcavins
     Type:  enhancement             |      Status:  assigned
 Priority:  normal                  |   Milestone:  2.7
Component:  Groups                  |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  dev-feedback has-patch  |
------------------------------------+-----------------------
Changes (by DJPaul):

 * cc: ubernaut@… (added)


Comment:

 The patch is looking really close. Question for @r-a-y:

 The `populate` change you made to invalidate old caches. Doing this here
 strikes me a little odd, because this is basically a one-time (per group)
 only thing. Once all the groups have been cleared, it'll never get used
 again.

 Did you consider fetching all Group IDs during the database upgrade
 routine, and clearing those there?
 We don't need to worry about re-building the cache there, and I'd even
 suggest a raw SQL query to fetch the IDs if `groups_get_groups` doesn't
 support such an optimal query (I haven't looked at its query builder for a
 while).

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