[buddypress-trac] [BuddyPress Trac] #7573: BP_Group_Member_Query should support multiple group IDs
buddypress-trac
noreply at wordpress.org
Wed Jul 26 01:03:08 UTC 2017
#7573: BP_Group_Member_Query should support multiple group IDs
-------------------------+------------------------------
Reporter: r-a-y | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Groups | Version: 1.8
Severity: normal | Resolution:
Keywords: has-patch |
-------------------------+------------------------------
Comment (by dcavins):
Replying to [comment:2 r-a-y]:
> @dcavins - You're right. The `else` condition is not necessary.
>
> I guess I was just being extra cautious, but `wp_parse_id_list()` should
handle most empty instances and default to `array( 0 )`, so we should be
safe to remove the `else` condition.
Not so fast! :)
I think we need an else condition, because `wp_parse_id_list( array() )`
does return an empty array. But maybe the else condition should be
explicitly not made to work, like `group_id = 0` or something. I was just
wondering, could there be a passed arg that would be useful? I can't think
of one.
Thanks again for this good idea!
--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7573#comment:3>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac
More information about the buddypress-trac
mailing list