From 75ab7e0114759b02858199420a14fa6d30fde107 Mon Sep 17 00:00:00 2001
From: "wurblzap%gmail.com" <>
Date: Tue, 3 Jan 2006 16:40:45 +0000
Subject: Bug 322082: Clean up user selection SQL. Patch by Marc Schumann
 <wurblzap@gmail.com>, r=LpSolit, a=justdave

---
 editusers.cgi | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

(limited to 'editusers.cgi')

diff --git a/editusers.cgi b/editusers.cgi
index 40854775f..3e23d0808 100755
--- a/editusers.cgi
+++ b/editusers.cgi
@@ -100,9 +100,15 @@ if ($action eq 'search') {
     } else {
         $visibleGroups = 1;
         if ($grouprestrict eq '1') {
-            $query .= ', user_group_map AS ugm';
+            $query .= qq{, user_group_map AS ugm
+                         WHERE ugm.user_id = profiles.userid
+                           AND ugm.isbless = 0
+                        };
+            $nextCondition = 'AND';
+        }
+        else {
+            $nextCondition = 'WHERE';
         }
-        $nextCondition = 'WHERE';
     }
 
     if (!$visibleGroups) {
@@ -137,9 +143,7 @@ if ($action eq 'search') {
         if ($grouprestrict eq '1') {
             my $grouplist = join(',',
                 @{Bugzilla::User->flatten_group_membership($group->id)});
-            $query .= " $nextCondition profiles.userid = ugm.user_id " .
-                      "AND ugm.group_id IN($grouplist) " .
-                      "AND ugm.isbless = 0";
+            $query .= " $nextCondition ugm.group_id IN($grouplist) ";
         }
         $query .= ' ORDER BY profiles.login_name';
 
-- 
cgit v1.2.3-24-g4f1b