summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Lawrence <dlawrence@mozilla.com>2013-05-04 01:12:53 +0200
committerDavid Lawrence <dlawrence@mozilla.com>2013-05-04 01:12:53 +0200
commit2f7d085d82c45225062be0c096163cbe636ad451 (patch)
tree72b4320e228515104859551e699bc21a2006644f
parentbd1b0779824f599d21304fd99fce5dd919310a2b (diff)
downloadbugzilla-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
-rw-r--r--Bugzilla/CGI.pm4
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