Bug 274397 - Editing a saved search should now work even after clicking "Show List" from a bug.

Patch by Colin S. Ogilvie <colin.ogilvie@gmail.com>
r=wurblzap, a=justdave

1 files changed, 10 insertions, 9 deletions

diff --git a/buglist.cgi b/buglist.cgi
index 3379aa70a..bdda404a3 100755
--- a/buglist.cgi
+++ b/buglist.cgi
@@ -132,10 +132,11 @@ if ($::FORM{'regetlastlist'}) {
     $cgi->cookie('BUGLIST') || ThrowUserError("missing_cookie");
 
     $order = "reuse last sort" unless $order;
-
+    my $bug_id = $cgi->cookie('BUGLIST');
+    $bug_id =~ s/:/,/g;
     # set up the params for this new query
     $params = new Bugzilla::CGI({
-                                 bug_id => [split(/:/, $cgi->cookie('BUGLIST'))],
+                                 bug_id => $bug_id,
                                  order => $order,
                                 });
 }
@@ -309,13 +310,6 @@ if ($::FORM{'cmdtype'} eq "dorem") {
         $params = new Bugzilla::CGI($::buffer);
         $order = $params->param('order') || $order;
 
-        # backward compatibility hack: if the saved query doesn't say which
-        # form was used to create it, assume it was on the advanced query
-        # form - see bug 252295
-        if (!$params->param('query_format')) {
-            $params->param('query_format', 'advanced');
-            $::buffer = $params->query_string;
-        }
     }
     elsif ($::FORM{'remaction'} eq "runseries") {
         $::buffer = LookupSeries($::FORM{"series_id"});
@@ -418,6 +412,13 @@ elsif (($::FORM{'cmdtype'} eq "doit") && $::FORM{'remtype'}) {
     }
 }
 
+# backward compatibility hack: if the saved query doesn't say which
+# form was used to create it, assume it was on the advanced query
+# form - see bug 252295
+if (!$params->param('query_format')) {
+    $params->param('query_format', 'advanced');
+    $::buffer = $params->query_string;
+}
 
 ################################################################################
 # Column Definition