Bug 383643: When using "Find a specific bug", the generated buglist does not let…

Bug 383643: When using "Find a specific bug", the generated buglist does not let you change the sort order - Patch by Alex Schuilenburg <alex@schuilenburg.org> r/a=LpSolit

Bug 383643: When using "Find a specific bug", the generated buglist does not let…
3566e55c · lpsolit%gmail.com · 6724564e · 3566e55c · 3566e55c · 3566e55c
Commit 3566e55c authored Jan 12, 2008 by lpsolit%gmail.com
Hide whitespace changes
Inline Side-by-side

Showing with 24 additions and 33 deletions

buglist.cgi buglist.cgi +1 -1

list.html.tmpl template/en/default/list/list.html.tmpl +0 -1

table.html.tmpl template/en/default/list/table.html.tmpl +23 -31

No files found.
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -965,7 +965,7 @@ if (defined $cgi->param('limit')) {
 }
 elsif ($fulltext) {
    $query .= " " . $dbh->sql_limit(FULLTEXT_BUGLIST_LIMIT);
-    $vars->{'sorted_by_relevance'} = 1;
+    $vars->{'message'} = 'buglist_sorted_by_relevance' if ($cgi->param('order') =~ /^relevance/);
 }

--- a/template/en/default/list/list.html.tmpl
+++ b/template/en/default/list/list.html.tmpl
@@ -36,7 +36,6 @@
 [% END %]
 [% qorder = order FILTER url_quote IF order %]
-[% message = "buglist_sorted_by_relevance" IF sorted_by_relevance %]
 [%############################################################################%]

--- a/template/en/default/list/table.html.tmpl
+++ b/template/en/default/list/table.html.tmpl
@@ -85,18 +85,14 @@
      <th>&nbsp;</th>
      [% END %]
      <th colspan="[% splitheader ? 2 : 1 %]" class="first-child">
-        [% IF sorted_by_relevance %]
+        [% desc = '' %]
-          ID
+        [% IF (om = order.match("^bugs\.bug_id( desc)?")) %]
-        [% ELSE %]
+          [% desc = ' desc' IF NOT om.0 %]
-          [% desc = '' %]
-          [% IF (om = order.match("^bugs\.bug_id( desc)?")) %]
-            [% desc = ' desc' IF NOT om.0 %]
-          [% END %]
-          <a href="buglist.cgi?
-                    [% urlquerypart FILTER html %]&amp;order=bugs.bug_id[% desc FILTER url_quote %]
-                    [%-#%]&amp;query_based_on=
-                    [% defaultsavename OR searchname FILTER url_quote %]">ID</a>
        [% END %]
+        <a href="buglist.cgi?
+                  [% urlquerypart FILTER html %]&amp;order=bugs.bug_id[% desc FILTER url_quote %]
+                  [%-#%]&amp;query_based_on=
+                  [% defaultsavename OR searchname FILTER url_quote %]">ID</a>
      </th>
      [% IF splitheader %]
@@ -133,28 +129,24 @@
 [% BLOCK columnheader %]
  <th colspan="[% splitheader ? 2 : 1 %]">
-    [% IF sorted_by_relevance %]
+    [% IF column.name.match('\s+AS\s+') %]
-      [%- abbrev.$id.title || field_descs.$id || column.title -%]
+      [%# For aliased columns, use their ID for sorting. %]
+      [% column.sortalias = id %]
    [% ELSE %]
-      [% IF column.name.match('\s+AS\s+') %]
+      [%# Other columns may sort on their name directly. %]
-        [%# For aliased columns, use their ID for sorting. %]
+      [% column.sortalias = column.name %]
-        [% column.sortalias = id %]
+    [% END %]
-      [% ELSE %]
+    [% desc = '' %]
-        [%# Other columns may sort on their name directly. %]
+    [% IF (om = order.match("$column.sortalias( desc)?")) %]
-        [% column.sortalias = column.name %]
+      [% desc = ' desc' IF NOT om.0 %]
-      [% END %]
-      [% desc = '' %]
-      [% IF (om = order.match("$column.sortalias( desc)?")) %]
-        [% desc = ' desc' IF NOT om.0 %]
-      [% END %]
-      [% order = order.remove("$column.sortalias( desc)?,?") %]
-      <a href="buglist.cgi?[% urlquerypart FILTER html %]&amp;order=
-        [% column.sortalias FILTER url_quote %][% desc FILTER url_quote %]
-        [% ",$order" FILTER url_quote IF order %]
-        [%-#%]&amp;query_based_on=
-        [% defaultsavename OR searchname FILTER url_quote %]">
-          [%- abbrev.$id.title || field_descs.$id || column.title -%]</a>
    [% END %]
+    [% order = order.remove("$column.sortalias( desc)?,?") %]
+    <a href="buglist.cgi?[% urlquerypart FILTER html %]&amp;order=
+      [% column.sortalias FILTER url_quote %][% desc FILTER url_quote %]
+      [% ",$order" FILTER url_quote IF order %]
+      [%-#%]&amp;query_based_on=
+      [% defaultsavename OR searchname FILTER url_quote %]">
+        [%- abbrev.$id.title || field_descs.$id || column.title -%]</a>
  </th>
 [% END %]