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

buddypress-trac noreply at wordpress.org
Thu Mar 23 14:56:51 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 dcavins):

 7477.2 and 7477.3 change `get()` to accept a slug, and then uses that
 change in `group_exists()`, as @johnjamesjacoby suggested. I like this
 approach, and don't see any downsides, unless I'm missing some performance
 effect.

 @boonebgorges, do have an opinion on the approach to use?

 Thanks for your feedback!

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


More information about the buddypress-trac mailing list