Commit c81a842c authored by Matt Tyson's avatar Matt Tyson Committed by Frédéric Buclin

Bug 1237161: Allow users with bless permissions to update users group membership using WebService

r=LpSolit a=dkl
parent 8c54443d
......@@ -326,6 +326,7 @@ sub update {
# Reject access if there is no sense in continuing.
$user->in_group('editusers')
|| $user->can_bless()
|| ThrowUserError("auth_failure", {group => "editusers",
action => "edit",
object => "users"});
......@@ -343,6 +344,8 @@ sub update {
delete $values->{ids};
$dbh->bz_start_transaction();
$values = { groups => $values->{groups} } unless $user->in_group('editusers');
foreach my $user (@$user_objects){
$user->set_all($values);
}
......
......@@ -275,6 +275,7 @@ sub update {
# Reject access if there is no sense in continuing.
$user->in_group('editusers')
|| $user->can_bless()
|| ThrowUserError("auth_failure", {group => "editusers",
action => "edit",
object => "users"});
......@@ -292,6 +293,8 @@ sub update {
delete $values->{ids};
$dbh->bz_start_transaction();
$values = { groups => $values->{groups} } unless $user->in_group('editusers');
foreach my $user (@$user_objects){
$user->set_all($values);
}
......@@ -709,7 +712,12 @@ B<EXPERIMENTAL>
=item B<Description>
Updates user accounts in Bugzilla.
Updates user accounts in Bugzilla. To use this method, you must be a member
of the C<editusers> group.
If you are not in the C<editusers> group, you may
add or remove users from groups if you have bless permissions for the groups
you wish to modify. All other changes will be ignored.
=item B<REST>
......
......@@ -162,6 +162,10 @@ Update User
Updates an existing user account in Bugzilla. You must be authenticated and be
in the *editusers* group to perform this action.
If you are not in the *editusers* group, you may add or remove users from groups
if you have bless permissions for the groups you wish to modify. All other changes
will be ignored.
**Request**
.. code-block:: text
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment