Bug 251669: add an option to show users in a drop down menu instead of a text edit field

patch by glob <bugzilla@glob.com.au>
r=joel
a=justdave

5 files changed, 96 insertions, 16 deletions

diff --git a/template/en/default/bug/create/create.html.tmpl b/template/en/default/bug/create/create.html.tmpl
index 3364649e8..ecb3dd8c9 100644
--- a/template/en/default/bug/create/create.html.tmpl
+++ b/template/en/default/bug/create/create.html.tmpl
@@ -185,8 +185,12 @@ function set_assign_to() {
       </strong>
     </td>
     <td colspan="3">
-      <input name="assigned_to" size="32" 
-             value="[% assigned_to FILTER html %]">
+      [% INCLUDE global/userselect.html.tmpl
+         name => "assigned_to"
+         value => assigned_to
+         size => 32
+         emptyok => 1
+       %]
       <noscript>(Leave blank to assign to default component owner)</noscript>
     </td>
   </tr>
@@ -194,7 +198,12 @@ function set_assign_to() {
   <tr>
     <td align="right"><strong>Cc:</strong></td>
     <td colspan="3">
-      <input name="cc" size="45" value="[% cc FILTER html %]">
+      [% INCLUDE global/userselect.html.tmpl
+         name => "cc"
+         value => cc
+         size => 45
+         emptyok => 1
+       %]
     </td>
   </tr>
   
diff --git a/template/en/default/bug/edit.html.tmpl b/template/en/default/bug/edit.html.tmpl
index ef6b551fa..f4c68bb4d 100644
--- a/template/en/default/bug/edit.html.tmpl
+++ b/template/en/default/bug/edit.html.tmpl
@@ -163,7 +163,13 @@
         <b><u>A</u>dd&nbsp;CC:</b>
       </td>
       <td>
-        <input name="newcc" size="30" value="" accesskey="a">
+         [% INCLUDE global/userselect.html.tmpl
+            name => "newcc"
+            value => ""
+            accesskey => "a"
+            size => 30
+            emptyok => 1
+          %]
       </td>
     </tr>
 
@@ -270,8 +276,13 @@
        <b><u>Q</u>A Contact:</b>
        </td>
        <td colspan="7">
-         <input name="qa_contact" accesskey="q"
-                value="[% bug.qa_contact.email FILTER html %]" size="60">
+         [% INCLUDE global/userselect.html.tmpl
+            name => "qa_contact"
+            value => bug.qa_contact.email
+            accesskey => "q"
+            size => 60
+            emptyok => 1
+          %]
        </td>
      </tr>
    [% END %]
diff --git a/template/en/default/bug/knob.html.tmpl b/template/en/default/bug/knob.html.tmpl
index b93f8099d..8ab0ce0ad 100644
--- a/template/en/default/bug/knob.html.tmpl
+++ b/template/en/default/bug/knob.html.tmpl
@@ -96,12 +96,15 @@
         <a href="page.cgi?id=fields.html#assigned_to">Reassign</a> 
         [% terms.bug %] to
       </label>
-      <input name="assigned_to" size="32"
-             onchange="if ((this.value != '[% bug.assigned_to.email FILTER js FILTER html %]') &&
-                            (this.value != '')) {
-                         document.changeform.knob[[% knum %]].checked=true;
-                       }"
-             value="[% bug.assigned_to.email FILTER html %]">
+      [% safe_assigned_to = FILTER js; bug.assigned_to.email; END %]
+      [% INCLUDE global/userselect.html.tmpl
+           name => "assigned_to"
+           value => bug.assigned_to.email
+           size => 32
+           onchange => "if ((this.value != '$safe_assigned_to') && (this.value != '')) {
+                          document.changeform.knob[$knum].checked=true;
+                        }"
+      %]
       <br>
       [% IF bug.isunconfirmed && bug.user.canconfirm %]
         &nbsp;&nbsp;&nbsp;&nbsp;<input type="checkbox" id="andconfirm" name="andconfirm">
diff --git a/template/en/default/global/userselect.html.tmpl b/template/en/default/global/userselect.html.tmpl
new file mode 100644
index 000000000..d83a5905b
--- /dev/null
+++ b/template/en/default/global/userselect.html.tmpl
@@ -0,0 +1,55 @@
+[%# 1.0@bugzilla.org %]
+[%# The contents of this file are subject to the Mozilla Public
+  # License Version 1.1 (the "License"); you may not use this file
+  # except in compliance with the License. You may obtain a copy of
+  # the License at http://www.mozilla.org/MPL/
+  #
+  # Software distributed under the License is distributed on an "AS
+  # IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or
+  # implied. See the License for the specific language governing
+  # rights and limitations under the License.
+  #
+  # The Original Code is the Bugzilla Bug Tracking System.
+  #
+  # Contributor(s): Byron Jones <bugzilla@glob.com.au>
+  #%]
+
+[%# INTERFACE:
+  # userlist: select only; array reference with list of users and identities
+  #                        userlist is built by Bugzilla::User::get_userlist()
+  # name: mandatory; field name
+  # value: optional; default field value/selection
+  # onchange: optional; onchange attribute value
+  # accesskey: optional, input only; accesskey attribute value
+  # size: optional, input only; size attribute value
+  # emptyok: optional, select only;  if true, prepend menu option to start of select
+  #
+  #%]
+
+[% IF Param("usemenuforusers") %]
+<select name="[% name FILTER html %]"
+  [% IF onchange %] onchange="[% onchange FILTER html %]" [% END %]
+  [% IF accesskey %] accesskey="[% accesskey FILTER html %]" [% END %]
+>
+  [% IF emptyok %]
+    <option value=""></option>
+  [% END %]
+  [% FOREACH tmpuser = user.get_userlist %]
+    [% IF tmpuser.visible OR value == tmpuser.login %]
+      <option value="[% tmpuser.login FILTER html %]"
+        [% " selected" IF value == tmpuser.login %]
+      >[% tmpuser.identity FILTER html %]</option>
+    [% END %]
+  [% END %]
+</select>
+[% ELSE %]
+<input
+  name="[% name FILTER html %]"
+  value="[% value FILTER html %]"
+  [% IF accesskey %] accesskey="[% accesskey FILTER html %]" [% END %]
+  [% IF size %] size="[% size FILTER html %]" [% END %]
+  [% IF onchange %] onchange="[% onchange FILTER html %]" [% END %]
+>
+[% END %]
+
+
diff --git a/template/en/default/list/edit-multiple.html.tmpl b/template/en/default/list/edit-multiple.html.tmpl
index f03ea09e1..2f61ae45b 100644
--- a/template/en/default/list/edit-multiple.html.tmpl
+++ b/template/en/default/list/edit-multiple.html.tmpl
@@ -302,10 +302,12 @@
 <label for="knob-reassign"><a href="page.cgi?id=fields.html#assigned_to">
   Reassign</a> [% terms.bugs %] to
 </label>
-<input name="assigned_to"
-       value="[% user.login FILTER html %]"
-       onchange="document.forms.changeform.knob[[% knum %]].checked = true;"
-       size="32"><br>
+[% INCLUDE global/userselect.html.tmpl
+   name => "assigned_to"
+   value => user.login
+   size => 32
+   onchange => "document.forms.changeform.knob[$knum].checked=true;"
+%]<br>
 
 [% knum = knum + 1 %]
 <input id="knob-reassignbycomponent"