[buddypress-trac] [BuddyPress Trac] #7477: BP_Groups_Group::group_exists() should be cached.

buddypress-trac noreply at wordpress.org
Wed Mar 29 20:43:05 UTC 2017


#7477: BP_Groups_Group::group_exists() should be cached.
------------------------------------+-----------------------
 Reporter:  dcavins                 |       Owner:  dcavins
     Type:  enhancement             |      Status:  accepted
 Priority:  normal                  |   Milestone:  2.9
Component:  Groups                  |     Version:  2.8.2
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+-----------------------

Comment (by r-a-y):

 I'm concerned about the cache purging strategy when we use
 `bp_core_get_incremented_cache()` to fetch the group cache, especially
 when working with the edit group slug ticket (#6014).

 If group A changes their slug from `old-slug` to `new-slug` and Group B is
 created with the slug `old-slug`.  Wouldn't using
 `BP_Groups_Group::group_exists( 'old-slug' )` return Group A due to how
 the cache reference due to how the cache key in
 `bp_core_get_incremented_cache()` works?

 How does cache invalidation work here?

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


More information about the buddypress-trac mailing list