From: lpsolit%gmail.com <>
Date: Tue, 30 May 2006 02:15:56 +0000 (+0000)
Subject: Bug 335790: Field names in search form lists should be ordered alphabetically - Patch... 
X-Git-Tag: bugzilla-2.23.2~148
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f3254c63caf96f5a1106272d8393a66f4a32fc42;p=thirdparty%2Fbugzilla.git

Bug 335790: Field names in search form lists should be ordered alphabetically - Patch by Toms Baugis <toms.baugis@gmail.com> r=wicked a=myk
---

diff --git a/query.cgi b/query.cgi
index 8afb327e93..daae265d72 100755
--- a/query.cgi
+++ b/query.cgi
@@ -282,6 +282,7 @@ $vars->{'bug_severity'} = \@::legal_severity;
 my @fields;
 push(@fields, { name => "noop", description => "---" });
 push(@fields, $dbh->bz_get_field_defs());
+@fields = sort {lc($a->{'description'}) cmp lc($b->{'description'})} @fields;
 $vars->{'fields'} = \@fields;
 
 # Creating new charts - if the cmd-add value is there, we define the field
diff --git a/template/en/default/search/form.html.tmpl b/template/en/default/search/form.html.tmpl
index c3b4d8bcb6..879625d02c 100644
--- a/template/en/default/search/form.html.tmpl
+++ b/template/en/default/search/form.html.tmpl
@@ -589,11 +589,17 @@ function doOnSelectProduct(selectmode) {
     <label for="chfield">where one or more of the following changed</label>:
   </dt>
   <dd>
-    <select name="chfield" id="chfield" multiple="multiple" size="4">
+    [%# Create array, so we can sort it by description #%]
+    [% chfields = [] %]
     [% FOREACH field = chfield %]
-      <option value="[% field FILTER html %]"
-        [% " selected" IF lsearch(default.chfield, field) != -1 %]>
-        [% (field_descs.$field || field) FILTER html %]</option>
+      [% chfields.push({value => field, desc => (field_descs.$field || field) }) %]
+    [% END %]
+
+    <select name="chfield" id="chfield" multiple="multiple" size="4">
+    [% FOREACH field = chfields.sort('desc') %]
+      <option value="[% field.value FILTER html %]"
+        [% " selected" IF lsearch(default.chfield, field.value) != -1 %]>
+        [% field.desc FILTER html %]</option>
     [% END %]
     </select>
   </dd>