Bug 174112 Edit multiple bugs broken

r=burnus

diff --git a/buglist.cgi b/buglist.cgi
index 4ad0720a93e37b862b9f9ed48ff2e5c70e2e3695..9f00e15aeab1098a8895dea821e9db28780814bf 100755 (executable)
--- 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 18f18e4743a8d9056cb0fbde1a4b31656ee7cf8f..f529f13ea9358cf045778d31f582b46bb34e8c59 100755 (executable)
--- 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 a9088a62d829962be23d9c22b050ff61d9fd4684..4121d029252ac7f2e90bd6f15cc6921b8eecf95f 100644 (file)
--- 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>