Missing break statement inside bp_groups_admin_load() function

[tw2113]

The "mod" case section is missing a break statement.

[imath]

If you add the break statement, then it's no more possible to promote a member as a moderator of the group from the administration screen.

So this ticket needs a deeper investigation, we shouldn't touch anything about it in 2.3.

[sourabh.matolia]

Hello,

Looks like It is intentional that we don't have a 'break' statement under
function 'bp_groups_admin_load' > switch ( $new_role ) > case 'mod'.

case 'mod' > Simply change the user role to 'member', next it gets into the case 'admin'

case 'admin' > Updates the new role to mod (Moderator)

I think we can close the ticket.