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

[buddypress-trac]

#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  |
------------------------------------+-----------------------
Changes (by dcavins):

 * owner:   => dcavins
 * status:  new => accepted


Comment:

 @johnjamesjacoby I like the way you think, but, alas, the `get()` method
 doesn't have a `slug` option. We could add it to `get()` and then we'd
 also have caching.

 I have no idea why there's a table name parameter. So people could use a
 separate and super fast table for looking up groups?

 The table name parameter would have to be deprecated if we moved to using
 `get()` internally, if that's an issue.

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