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

buddypress-trac noreply at wordpress.org
Wed Mar 29 21:22:52 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):

 >  As far as I understand, the cache incrementor is reset when the
 groups_group_after_save action is called, so queries after a group is
 updated will miss the cache and need to be re-run since the group table or
 meta table has been changed in a meaningful way.

 Yeah, you're right.  For some reason, I thought the group cache
 invalidation was less of a sledgehammer, but it works in this context! :)

 That was my main hang-up.  The rest of the patch looks good to me.

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

More information about the buddypress-trac mailing list