From: bugreport%peshkin.net <>
Date: Sun, 13 Oct 2002 00:00:29 +0000 (+0000)
Subject: Bug 174112 Edit multiple bugs broken
X-Git-Tag: bugzilla-2.17.1~70
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f61593bee73b37fc12caabbb2958b6515d688420;p=thirdparty%2Fbugzilla.git

Bug 174112 Edit multiple bugs broken
r=burnus
---

diff --git a/buglist.cgi b/buglist.cgi
index 4ad0720a93..9f00e15aea 100755
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -200,12 +200,12 @@ sub GetGroupsByUserId {
     return if !$userid;
 
     SendSQL("
-        SELECT  groups.id, name, description, isactive
-          FROM  groups, user_group_map
-         WHERE  user_id = $userid AND NOT isbless
-           AND  user_group_map.group_id = groups.id
-           AND  isbuggroup
-      ORDER BY  description ");
+        SELECT DISTINCT  groups.id, name, description, isactive
+                   FROM  groups, user_group_map
+                  WHERE  user_id = $userid AND NOT isbless
+                    AND  user_group_map.group_id = groups.id
+                    AND  isbuggroup
+               ORDER BY  description ");
 
     my @groups;
 
diff --git a/process_bug.cgi b/process_bug.cgi
index 18f18e4743..f529f13ea9 100755
--- a/process_bug.cgi
+++ b/process_bug.cgi
@@ -931,6 +931,7 @@ if ($::FORM{'keywords'}) {
 my $keywordaction = $::FORM{'keywordaction'} || "makeexact";
 
 if ($::comma eq ""
+    && (! @groupAdd) && (! @groupDel)
     && (! @::legal_keywords || (0 == @keywordlist && $keywordaction ne "makeexact"))
     && defined $::FORM{'masscc'} && ! $::FORM{'masscc'}
     ) {
diff --git a/template/en/default/list/edit-multiple.html.tmpl b/template/en/default/list/edit-multiple.html.tmpl
index a9088a62d8..4121d02925 100644
--- a/template/en/default/list/edit-multiple.html.tmpl
+++ b/template/en/default/list/edit-multiple.html.tmpl
@@ -182,14 +182,14 @@
     [% FOREACH group = groups %]
     <tr>
       <td align="center">
-        <input type="radio" name="bit-[% group.bit %]" value="-1" checked="checked">
+        <input type="radio" name="bit-[% group.id %]" value="-1" checked="checked">
       </td>
       <td align="center">
-        <input type="radio" name="bit-[% group.bit %]" value="0">
+        <input type="radio" name="bit-[% group.id %]" value="0">
       </td>
       [% IF group.isactive %]
         <td align="center">
-          <input type="radio" name="bit-[% group.bit %]" value="1">
+          <input type="radio" name="bit-[% group.id %]" value="1">
         </td>
       [% ELSE %]
         <td>&nbsp;</td>