Bug 288603: Allow users in the request group to remove pending requests made by others - Patch by Fr�d�ric Buclin <LpSolit@gmail.com> r=myk a=justdave

webtools/bugzilla/Bugzilla/Flag.pm

@@ -320,8 +320,11 @@ sub validate {
         # - The flag is unchanged
         next if ($status eq $flag->{status});
 
-        # - User can clear flags set by itself
+        # - User in the $request_gid group can clear pending requests
-        next if (($status eq "X") && ($user->id eq $flag->{setter}->id));
+        next if ($status eq 'X'
+                 && $flag->{status} eq '?'
+                 && (!$flag->{type}->{request_gid}
+                     || $user->in_group(&::GroupIdToName($flag->{type}->{request_gid}))));
 
         # - User in the $grant_gid group can set/clear flags,
         #   including "+" and "-"