[buddypress-trac] [BuddyPress Trac] #5733: Use wp_cache_add_global_groups() so cache is applicable throughout multisite

[buddypress-trac]

#5733: Use wp_cache_add_global_groups() so cache is applicable throughout
multisite
--------------------------+-----------------------------
 Reporter:  wpdennis      |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Future Release
Component:  Core          |     Version:  2.0
 Severity:  minor         |  Resolution:
 Keywords:  2nd-opinion   |
--------------------------+-----------------------------

Comment (by boonebgorges):

 >  Looks okay, but instead of doing a find and replace for 'bp'
 everywhere, perhaps we should create wrapper wp_cache_X() functions and
 replace all wp_cache_X() calls with bp_cache_X(). This is so we just have
 to change the global cache group in one place.

 But most of our caching doesn't actually use the global group - it uses
 object-type-specific cache groups (like 'bp_activity'). Creating a wrapper
 in one place suggests creating a wrapper everywhere, and I don't want to
 go wrapper-crazy without good argument :)

 >  The BP Multi Network plugin also reminds me of how our notification
 meta keys suck

 Truth.

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