From 73fd49ff3bbff6244802ba548bb22c2be39014e1 Mon Sep 17 00:00:00 2001
From: "bugreport%peshkin.net" <>
Date: Tue, 6 Jul 2004 08:12:29 +0000
Subject: Bug 243463 Use a param to protect new charts from leaking information
r=justdave a=justdave
---
editproducts.cgi | 59 ++++++++++++++++++++++++++++++--------------------------
1 file changed, 32 insertions(+), 27 deletions(-)
(limited to 'editproducts.cgi')
diff --git a/editproducts.cgi b/editproducts.cgi
index 8cf9a309d..6d33c8080 100755
--- a/editproducts.cgi
+++ b/editproducts.cgi
@@ -271,6 +271,10 @@ if ($action eq 'add') {
print "
\n";
print "
Version:
\n";
print "
\n";
+ print "
\n";
+ print "
Create chart datasets for this product:
\n";
+ print "
";
+ print "
\n";
print "\n\n";
print "\n";
@@ -389,36 +393,37 @@ if ($action eq 'new') {
CONTROLMAPNA . ", 0)");
}
- # Insert default charting queries for this product.
- # If they aren't using charting, this won't do any harm.
- GetVersionTable();
-
- my @series;
-
- # 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"]);
- }
+ if ($::FORM{createseries}) {
+ # Insert default charting queries for this product.
+ # If they aren't using charting, this won't do any harm.
+ GetVersionTable();
+
+ my @series;
+
+ # 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"]);
+ }
- foreach my $resolution (@::legal_resolution) {
- next if !$resolution;
- push(@series, [$resolution, "resolution=$resolution"]);
- }
+ foreach my $resolution (@::legal_resolution) {
+ next if !$resolution;
+ push(@series, [$resolution, "resolution=$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);
- 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);
- $series->writeToDatabase();
+ # 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);
+ 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);
+ $series->writeToDatabase();
+ }
}
-
# Make versioncache flush
unlink "$datadir/versioncache";
--
cgit v1.2.3-24-g4f1b