From: David Lawrence <dlawrence@mozilla.com>
Date: Fri, 3 May 2013 23:16:58 +0000 (-0400)
Subject: Bug 652625 - Empty queries still get run because the list_id parameter is added to... 
X-Git-Tag: bugzilla-4.0.2~47
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0cf288f0b0f64c459bd75731559671a6f2c6d306;p=thirdparty%2Fbugzilla.git

Bug 652625 - Empty queries still get run because the list_id parameter is added to them
r/a=mkanat
---

diff --git a/Bugzilla/CGI.pm b/Bugzilla/CGI.pm
index 106944d74a..b07e356866 100644
--- a/Bugzilla/CGI.pm
+++ b/Bugzilla/CGI.pm
@@ -440,7 +440,9 @@ sub redirect_search_url {
 
     $self->clean_search_url();
 
-    if ($user->id) {
+    # Make sure we 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