From f9214d522c9dabc0096573e553b946c492a65230 Mon Sep 17 00:00:00 2001 From: "gerv%gerv.net" <> Date: Mon, 30 Aug 2004 04:29:34 +0000 Subject: Bug 225687 - add group controls to charts, along with various other cleanups. Patch by gerv; r=joel, a=justdave. --- editproducts.cgi | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'editproducts.cgi') diff --git a/editproducts.cgi b/editproducts.cgi index 74a62166e..873db1afc 100755 --- a/editproducts.cgi +++ b/editproducts.cgi @@ -557,25 +557,27 @@ if ($action eq 'new') { # We do every status, every resolution, and an "opened" one as well. foreach my $bug_status (@::legal_bug_status) { - push(@series, [$bug_status, "bug_status=$bug_status"]); + push(@series, [$bug_status, + "bug_status=" . url_quote($bug_status)]); } foreach my $resolution (@::legal_resolution) { next if !$resolution; - push(@series, [$resolution, "resolution=$resolution"]); + push(@series, [$resolution, "resolution=" .url_quote($resolution)]); } # For localisation reasons, we get the name of the "global" subcategory # and the title of the "open" query from the submitted form. - my @openedstatuses = ("UNCONFIRMED", "NEW", "ASSIGNED", "REOPENED"); - my $query = join("&", map { "bug_status=$_" } @openedstatuses); + my @openedstatuses = OpenStates(); + my $query = + join("&", map { "bug_status=" . url_quote($_) } @openedstatuses); push(@series, [$::FORM{'open_name'}, $query]); foreach my $sdata (@series) { my $series = new Bugzilla::Series(undef, $product, - $::FORM{'subcategory'}, - $sdata->[0], $::userid, 1, - $sdata->[1] . "&product=$product", 1); + $::FORM{'subcategory'}, + $sdata->[0], $::userid, 1, + $sdata->[1] . "&product=" . url_quote($product), 1); $series->writeToDatabase(); } } -- cgit v1.2.3-24-g4f1b