[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