[buddypress-trac] [BuddyPress] #3631: Groups Bugs

[buddypress-trac at lists.automattic.com]

#3631: Groups Bugs
--------------------+-----------------------
 Reporter:  ken_k   |       Owner:  DJPaul
     Type:  defect  |      Status:  accepted
 Priority:  normal  |   Milestone:  1.5.1
Component:  Groups  |     Version:  1.5
 Severity:  normal  |  Resolution:
 Keywords:          |
--------------------+-----------------------

Comment (by DJPaul):

 Remaining issue is how to deal with existing groups that have no admins;
 the log dumps errors like:

 {{{
 [30-Sep-2011 21:34:23] WordPress database error You have an error in your
 SQL syntax; check the manual that corresponds to your MySQL server version
 for the right syntax to use near ')' at line 1 for query SELECT
 COUNT(DISTINCT ID) FROM wp_users WHERE user_status = 0 AND ID IN (  )
 made by require, wp, WP->main, do_action_ref_array, call_user_func_array,
 bp_screens, do_action, call_user_func_array,
 groups_screen_group_admin_manage_members, bp_core_load_template,
 load_template, require_once, locate_template, load_template, require_once,
 bp_has_members, BP_Core_Members_Template->__construct,
 BP_Core_User->get_specific_users
 }}}

 $include in BP_Core_User->get_specific_users() is an empty string, causing
 the false check to be handled incorrectly (line 184ish, bp-members-
 template.php). I see 3 options to fix: 1) Update this query logic, 2)
 Update the 'group admin manage members' to check we don't pass an empty
 string to get_specific_users(), or 3) don't do anything as this is an edge
 case caused by a (now fixed) bug.

 I think option 3 is appropriate but appreciate input from other devs.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3631#comment:6>
BuddyPress <http://buddypress.org/>
BuddyPress