From 3e087f1504c17f43b0e4f278527247d67de975b4 Mon Sep 17 00:00:00 2001 From: "justdave%bugzilla.org" Date: Wed, 7 Jul 2004 06:02:33 +0000 Subject: [PATCH] Bug 245077: The "Find a specific bug" tab is now the default query when you go to query.cgi. To keep the developers happy, a cookie is now set to remember which query page you last visited, and going to query.cgi without specifying which tab will get you the last one you visited. Patch by Byron Jones r=justdave, a=justdave git-svn-id: svn://10.0.0.236/trunk@158831 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/webtools/bugzilla/query.cgi | 25 ++++- .../en/default/search/search.html.tmpl | 97 ------------------- .../template/en/default/search/tabs.html.tmpl | 10 +- 3 files changed, 25 insertions(+), 107 deletions(-) delete mode 100644 mozilla/webtools/bugzilla/template/en/default/search/search.html.tmpl diff --git a/mozilla/webtools/bugzilla/query.cgi b/mozilla/webtools/bugzilla/query.cgi index a37fc91256c..c97c07eed0b 100755 --- a/mozilla/webtools/bugzilla/query.cgi +++ b/mozilla/webtools/bugzilla/query.cgi @@ -22,6 +22,7 @@ # David Gardiner # Matthias Radestock # Gervase Markham +# Byron Jones use strict; use lib "."; @@ -395,16 +396,32 @@ if (($::FORM{'query_format'} || $::FORM{'format'} || "") eq "create-series") { # Add in the defaults. $vars->{'default'} = \%default; -$vars->{'format'} = $::FORM{'format'}; -$vars->{'query_format'} = $::FORM{'query_format'}; +$vars->{'format'} = $cgi->param('format'); +$vars->{'query_format'} = $cgi->param('query_format'); + +# Set default page to "specific" if none proviced +if (!($cgi->param('query_format') || $cgi->param('format'))) { + if (defined $cgi->cookie('DEFAULTFORMAT')) { + $vars->{'format'} = $cgi->cookie('DEFAULTFORMAT'); + } else { + $vars->{'format'} = 'specific'; + } +} + +# Set cookie from format unless it's a report +if ($vars->{'format'} !~ /^report-/i) { + $cgi->send_cookie(-name => 'DEFAULTFORMAT', + -value => $vars->{'format'}, + -expires => "Fri, 01-Jan-2038 00:00:00 GMT"); +} # Generate and return the UI (HTML page) from the appropriate template. # 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'}); + $vars->{'query_format'} || $vars->{'format'}, + $cgi->param('ctype')); print $cgi->header($format->{'ctype'}); diff --git a/mozilla/webtools/bugzilla/template/en/default/search/search.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/search/search.html.tmpl deleted file mode 100644 index bbed6c2d6ea..00000000000 --- a/mozilla/webtools/bugzilla/template/en/default/search/search.html.tmpl +++ /dev/null @@ -1,97 +0,0 @@ -[%# 1.0@bugzilla.org %] -[%# The contents of this file are subject to the Mozilla Public - # License Version 1.1 (the "License"); you may not use this file - # except in compliance with the License. You may obtain a copy of - # the License at http://www.mozilla.org/MPL/ - # - # Software distributed under the License is distributed on an "AS - # IS" basis, WITHOUT WARRANTY OF ANY KIND, either express or - # implied. See the License for the specific language governing - # rights and limitations under the License. - # - # The Original Code is the Bugzilla Bug Tracking System. - # - # The Initial Developer of the Original Code is Netscape Communications - # Corporation. Portions created by Netscape are - # Copyright (C) 1998 Netscape Communications Corporation. All - # Rights Reserved. - # - # Contributor(s): Gervase Markham - #%] - -[%# INTERFACE: - # This template has no interface. However, to use it, you need to fulfill - # the interfaces of search/form.html.tmpl, search/knob.html.tmpl and - # search/boolean-charts.html.tmpl. - #%] - -[% PROCESS global/variables.none.tmpl %] - -[% USE Bugzilla %] -[% cgi = Bugzilla.cgi %] - -[% PROCESS global/header.html.tmpl - title = "Search for $terms.bugs" - h1 = "" - onload = "selectProduct(document.forms['queryform']);initHelp();" - style = "td.selected_tab { - border-width: 2px 2px 0px; - border-style: solid; - } - td.unselected_tab, td.spacer { - border-width: 0px 0px 2px 0px; - border-style: solid; - } - dl.bug_changes dt { - margin-top: 15px; - }" -%] - -[% PROCESS search/tabs.html.tmpl %] - -[% button_name = "Search" %] - -[%# The decent help requires Javascript %] -[% IF NOT help %] - [% IF cgi.user_agent("Mozilla/5") %] - - - [% ELSE %] -

Give me help with this form.

- [% END %] -[% ELSE %] -

- For help, mouse over the page elements. - - [% IF cgi.user_agent("Mozilla/5") %] - Note that if the help popups are hidden by form element scroll bars, - this is a bug in your browser, - not in [% terms.Bugzilla %]. - [% END %] - -

-[% END %] -
- -[% PROCESS search/form.html.tmpl %] - -[% PROCESS search/knob.html.tmpl %] - -
- -[% PROCESS "search/boolean-charts.html.tmpl" %] - -

- Give me a clue about how to use this form. -

- -
- -[% PROCESS "search/search-help.html.tmpl" IF help %] - -[% PROCESS global/footer.html.tmpl %] diff --git a/mozilla/webtools/bugzilla/template/en/default/search/tabs.html.tmpl b/mozilla/webtools/bugzilla/template/en/default/search/tabs.html.tmpl index ac3c72f2fd7..06b22f7a282 100644 --- a/mozilla/webtools/bugzilla/template/en/default/search/tabs.html.tmpl +++ b/mozilla/webtools/bugzilla/template/en/default/search/tabs.html.tmpl @@ -26,10 +26,10 @@ # description: string. Description of the tab (used in tab title). #%] -[% tabs = [ { name => '__DEFAULT__', description => "Advanced Search" }, - { name => 'specific', description => "Find a Specific $terms.Bug " } ] %] +[% tabs = [ { name => 'specific', description => "Find a Specific $terms.Bug " }, + { name => 'advanced', description => "Advanced Search" } ] %] -[% current_tab = query_format || format || "__DEFAULT__" %] +[% current_tab = query_format || format || "advanced" %]
@@ -43,9 +43,7 @@ [% ELSE %]
- + [% tab.description %]