Bug 252856: HTML charts don't validate - Patch by victory(_RSZ_) <bmo@rsz.jp> r=gerv a=myk

template/en/default/filterexceptions.pl

@@ -123,7 +123,6 @@
 ],
 
 'reports/report-table.html.tmpl' => [
-  'buglistbase', 
   '"&$tbl_vals" IF tbl_vals', 
   '"&$col_vals" IF col_vals', 
   '"&$row_vals" IF row_vals', 

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

@@ -37,10 +37,10 @@
 [% row_field_disp = field_descs.$row_field || row_field %]
   
 [% IF tbl == "-total-" %]
-  [% urlbase = BLOCK %]buglist.cgi?[% buglistbase %]
+  [% urlbase = BLOCK %]buglist.cgi?[% buglistbase FILTER html %]
   [% "&$tbl_vals" IF tbl_vals %][% END %]
 [% ELSE %]
-  [% urlbase = BLOCK %]buglist.cgi?[% buglistbase %]&
+  [% urlbase = BLOCK %]buglist.cgi?[% buglistbase FILTER html %]&
   [% tbl_field FILTER url_quote %]=[% tbl FILTER url_quote %][% END %]
 [% END %]
 
@@ -154,7 +154,6 @@
           [% col_field FILTER url_quote %]=[% col FILTER url_quote %]
           [% "&$row_vals" IF row_vals %]">
         [% col_totals.$col %]</a>
-      <strong>
       </td> 
     [% END %]
     <td class="ttotal" align="right">

template/en/default/reports/report.html.tmpl

@@ -51,6 +51,7 @@
 [% tbl_field_disp = field_descs.$tbl_field || tbl_field %]
 [% col_field_disp = field_descs.$col_field || col_field %]
 [% row_field_disp = field_descs.$row_field || row_field %]
+[% switchbase     = switchbase FILTER html %]
 
 [% title = BLOCK %]
   Report: 
@@ -125,8 +126,8 @@
                        { name => "line",  description => "Line" },
                        { name => "table", description => "Table" } ] %]
 
-        [% formaturl = "report.cgi?$switchbase&width=$width&height=$height" _ 
-                       "&action=wrap" %]
+        [% formaturl = "report.cgi?$switchbase&width=$width" _ 
+                       "&height=$height&action=wrap" %]
         [% FOREACH other_format = formats %]
           [% NEXT IF other_format.name == "pie" AND row_field AND col_field %]
           [% UNLESS other_format.name == format %]
@@ -157,7 +158,7 @@
                    [% height - 100 %]">Shorter</a><br>
         </td>
       [% END %]
-    <tr>
+    </tr>
   </table>
 
   <p>