diff options
| author | David Lawrence <dlawrence@mozilla.com> | 2013-05-04 01:12:53 +0200 |
|---|---|---|
| committer | David Lawrence <dlawrence@mozilla.com> | 2013-05-04 01:12:53 +0200 |
| commit | 2f7d085d82c45225062be0c096163cbe636ad451 (patch) | |
| tree | 72b4320e228515104859551e699bc21a2006644f /Bugzilla | |
| parent | bd1b0779824f599d21304fd99fce5dd919310a2b (diff) | |
| download | bugzilla-2f7d085d82c45225062be0c096163cbe636ad451.tar.gz bugzilla-2f7d085d82c45225062be0c096163cbe636ad451.tar.xz | |
Bug 652625 - Empty queries still get run because the list_id parameter is added to them
r/a=mkanat
Diffstat (limited to 'Bugzilla')
| -rw-r--r-- | Bugzilla/CGI.pm | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm index 3165cb003..40eab2a15 100644 --- a/Bugzilla/CGI.pm +++ b/Bugzilla/CGI.pm @@ -438,7 +438,9 @@ sub redirect_search_url { $self->clean_search_url(); - if ($user->id) { + # Make sure we still have params still after cleaning otherwise we + # do not want to store a list_id for an empty search. + if ($user->id && $self->param) { # Insert a placeholder Bugzilla::Search::Recent, so that we know what # the id of the resulting search will be. This is then pulled out # of the Referer header when viewing show_bug.cgi to know what |