Bug 275608: The group selection when editing flag types should be a listbox with (no group) meaning no restriction - Patch by Frédéric Buclin <LpSolit@gmail.com> r=GavinS a=myk

2 files changed, 21 insertions, 4 deletions

diff --git a/template/en/default/admin/flag-type/edit.html.tmpl b/template/en/default/admin/flag-type/edit.html.tmpl
index 8491a1e7f..6016fcfae 100644
--- a/template/en/default/admin/flag-type/edit.html.tmpl
+++ b/template/en/default/admin/flag-type/edit.html.tmpl
@@ -195,8 +195,8 @@
       <th>Grant Group:</th>
       <td>
         the group allowed to grant/deny flags of this type
-        (to allow all users to grant/deny these flags, leave this empty)<br>
-        <input type="text" name="grant_gid" value="[% type.grant_gid FILTER html %]" size="50" maxlength="255">
+        (to allow all users to grant/deny these flags, select no group)<br>
+        [% PROCESS select selname = "grant_gid" %]
       </td>
     </tr>
 
@@ -204,9 +204,9 @@
       <th>Request Group:</th>
       <td>
         if flags of this type are requestable, the group allowed to request them
-        (to allow all users to request these flags, leave this empty)<br>
+        (to allow all users to request these flags, select no group)<br>
         Note that the request group alone has no effect if the grant group is not defined!<br>
-        <input type="text" name="request_gid" value="[% type.request_gid FILTER html %]" size="50" maxlength="255">
+        [% PROCESS select selname = "request_gid" %]
       </td>
     </tr>
 
@@ -225,3 +225,19 @@
 </form>
 
 [% PROCESS global/footer.html.tmpl %]
+
+
+[%############################################################################%]
+[%# Block for SELECT fields                                                  #%]
+[%############################################################################%]
+
+[% BLOCK select %]
+  <select name="[% selname %]" id="[% selname %]">
+    <option value="">(no group)</option>
+    [% FOREACH group = groups %]
+      <option value="[% group.name FILTER html %]"
+        [% " selected" IF type.${selname} == group.name %]>[% group.name FILTER html %]
+      </option>
+    [% END %]
+  </select>
+[% END %]
diff --git a/template/en/default/filterexceptions.pl b/template/en/default/filterexceptions.pl
index 8ec9a3f3f..5d70380ca 100644
--- a/template/en/default/filterexceptions.pl
+++ b/template/en/default/filterexceptions.pl
@@ -563,6 +563,7 @@
   'type.sortkey || 1',
   'typeLabelLowerPlural',
   'typeLabelLowerSingular',
+  'selname',
 ],
 
 'admin/flag-type/list.html.tmpl' => [