Opened 10 years ago
Last modified 8 weeks ago
#6441 assigned defect (bug)
Missing break statement inside bp_groups_admin_load() function
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | normal | Version: | |
Component: | Groups | Keywords: | needs-patch close |
Cc: |
Description
The "mod" case section is missing a break statement.
Change History (4)
#2
@
10 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
#3
@
2 months ago
- Keywords close added
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.
Note: See
TracTickets for help on using
tickets.
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.