160476 - boolean chart addition doesn't keep query template format. Patch by gerv; r=myk.

query.cgi

@@ -362,8 +362,15 @@ $default{'querytype'} = $deforder || 'Importance';
 # Add in the defaults.
 $vars->{'default'} = \%default;
 
+$vars->{'format'} = $::FORM{'format'};
+
 # Generate and return the UI (HTML page) from the appropriate template.
-my $format = GetFormat("search/search", $::FORM{'format'}, $::FORM{'ctype'});
+# If we submit back to ourselves (for e.g. boolean charts), we need to
+# preserve format information; hence query_format taking priority over
+# format.
+my $format = GetFormat("search/search", 
+                       $::FORM{'query_format'} || $::FORM{'format'}, 
+                       $::FORM{'ctype'});
 print "Content-Type: $format->{'ctype'}\n\n";
 $template->process($format->{'template'}, $vars)
   || ThrowTemplateError($template->error());

template/en/default/search/search-advanced.html.tmpl

@@ -33,7 +33,9 @@
 [% button_name = "Search" %]
 
 <form method="get" action="buglist.cgi" name="queryform">
-
+  [%# If we resubmit to ourselves, we need to know if we are using a format. %]
+  <input type="hidden" name="query_format" value="[% format %]">
+  
 [% PROCESS search/form.html.tmpl %]
 
 [% PROCESS search/knob.html.tmpl %]

template/en/default/search/search.html.tmpl

@@ -33,7 +33,9 @@
 [% button_name = "Search" %]
 
 <form method="get" action="buglist.cgi" name="queryform">
-
+  [%# If we resubmit to ourselves, we need to know if we are using a format. %]
+  <input type="hidden" name="query_format" value="[% format %]">
+  
 [% PROCESS search/form.html.tmpl %]
 
 [% PROCESS search/knob.html.tmpl %]