From 6f4fba5980af8a0670aa402eb046fc58a06cb1b8 Mon Sep 17 00:00:00 2001
From: "lpsolit%gmail.com" <>
Date: Wed, 20 Apr 2005 01:11:41 +0000
Subject: 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
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 Bugzilla/Flag.pm | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'Bugzilla')

diff --git a/Bugzilla/Flag.pm b/Bugzilla/Flag.pm
index 29215876a..de9c3d23b 100644
--- a/Bugzilla/Flag.pm
+++ b/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
-        next if (($status eq "X") && ($user->id eq $flag->{setter}->id));
+        # - User in the $request_gid group can clear pending requests
+        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 "-"
-- 
cgit v1.2.3-24-g4f1b