From: Vladimir Panteleev <CyberShadow@users.noreply.github.com>
Date: Mon, 15 Mar 2021 14:50:43 +0000 (+0000)
Subject: table.html.tmpl: Use order_columns only when present (#63)
X-Git-Url: http://git.ipfire.org/gitweb/gitweb.cgi?a=commitdiff_plain;h=f0ed47ef0fe64ec55ccdd4fc881a033008c4e455;p=thirdparty%2Fbugzilla.git

table.html.tmpl: Use order_columns only when present (#63)

Fix a hard dependency on the BMO extension.

order_columns is set in Bugzilla::Extension::BMO, which might not be
present / loaded.

Co-authored-by: Dylan William Hardison <dylan@hardison.net>
---

diff --git a/template/en/default/list/table.html.tmpl b/template/en/default/list/table.html.tmpl
index 86aca78ed..d6c05c9e9 100644
--- a/template/en/default/list/table.html.tmpl
+++ b/template/en/default/list/table.html.tmpl
@@ -102,7 +102,9 @@
       <th class="sorttable_nosort">&nbsp;</th>
       [% END %]
       <th colspan="[% splitheader ? 2 : 1 %]" class="first-child
-                   sorted_[% lsearch(order_columns, 'bug_id') FILTER html %]">
+                   [% order_columns.defined
+                      ? 'sorted_' _ lsearch(order_columns, 'bug_id')
+                      : '' FILTER html %]">
         <a href="[% basepath FILTER none %]buglist.cgi?
                   [% urlquerypart FILTER html %]&amp;order=
                   [% PROCESS new_order id='bug_id' %]
@@ -148,7 +150,9 @@
 [% BLOCK columnheader %]
   <th colspan="[% splitheader ? 2 : 1 %]"
       class="sortable_column_[% key FILTER html %]
-             sorted_[% lsearch(order_columns, id) FILTER html %]">
+             [% order_columns.defined
+                ? 'sorted_' _ lsearch(order_columns, id)
+                : '' FILTER html %]">
     <a href="[% basepath FILTER none %]buglist.cgi?[% urlquerypart FILTER html %]&amp;order=
       [% PROCESS new_order %]
       [%-#%]&amp;query_based_on=