Bug 321691 duplicates.cgi doesn't use <th>

template/en/default/filterexceptions.pl

@@ -104,10 +104,6 @@
 ],
 
 'reports/duplicates-table.html.tmpl' => [
-  '"&maxrows=$maxrows" IF maxrows',
-  '"&changedsince=$changedsince" IF changedsince',
-  '"&bug_id=$bug_ids_string&sortvisible=1" 
-                                                            IF sortvisible',
   'column.name', 
   'column.description',
   'bug.id', 

template/en/default/reports/duplicates-table.html.tmpl

@@ -48,6 +48,7 @@
 
 [% IF bug_ids.size > 0 %]
   <table border>
+    <thead>
       <tr bgcolor="#CCCCCC">
         [% FOREACH column = [ { name => "id", description => "$terms.Bug #" },
                               { name => "count", description => "Dupe<br>Count" },
@@ -64,9 +65,7 @@
           [%# Small hack to keep delta column out if we don't need it %]
           [% NEXT IF column.name == "delta" AND NOT dobefore %]
 
-        <td>
-          <center>
-            <b>
+          <th>
             [% bug_ids_string = bug_ids.join(',') %]
             <a href="duplicates.cgi?sortby=[% column.name %]
               [% IF sortby == column.name %]
@@ -75,19 +74,17 @@
                 [%-# Some columns start off reversed %]
                 [% "&amp;reverse=1" IF column.name.match('delta|count') %]
               [% END %]
-                [% "&amp;maxrows=$maxrows" IF maxrows %]
-                [% "&amp;changedsince=$changedsince" IF changedsince %]
+              [% IF maxrows %]&amp;maxrows=[% maxrows FILTER html %][% END %]
+              [% IF changedsince %]&amp;changedsince=[% changedsince FILTER html %][% END %]
               [% "&amp;openonly=1" IF openonly %]
               [% FOREACH p = query_products %]&amp;product=[% p FILTER html %][% END %]
               [% IF format %]&amp;format=[% format FILTER html %][% END %]
-                [% "&amp;bug_id=$bug_ids_string&amp;sortvisible=1" 
-                                                            IF sortvisible %]">
+              [% IF sortvisible %]&amp;bug_id=[% bug_ids_string FILTER html %]&amp;sortvisible=1[% END %]">
               [% column.description %]</a>
-            </b>
-          </center>
-        </td>
+          </th>
         [% END %]
       </tr>
+    </thead>
 
     [% IF NOT sortby %]
       [% sortby = "count"; reverse = "1" %]
@@ -107,6 +104,7 @@
     [% END %]
 
     [%# *** Buglist *** %]
+    <tbody>
 
       [%# We need to keep track of the bug IDs we are actually displaying, because
         # if the user decides to sort the visible list, we need to know what that
@@ -142,7 +140,7 @@
           <td>[% bug.short_desc FILTER html %]</td>
         </tr>
       [% END %]
-
+    </tbody>
   </table>
 [% ELSE %]
   <h3>No duplicate [% terms.bugs %] found.</h3>