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

buddypress-trac noreply at wordpress.org
Thu Mar 30 02:35:04 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    |
-------------------------+-----------------------

Comment (by dcavins):

 Thanks, commenters!

 @boonebgorges, one specific question: I wrote the change to `get()` to
 accept a single slug argument as a string. Would you prefer that it accept
 an array or comma-separated string and use `IN()` SQL logic instead of `=`
 logic? I was imagining it for finding a group by slug, but I guess I can
 imagine the case where someone would want to find groups with `slug IN (
 'apples', 'oranges' )`. Either way seems OK to me. If you have a
 preference, let me know.

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


More information about the buddypress-trac mailing list