[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