From: lpsolit%gmail.com <>
Date: Sun, 4 Oct 2009 20:52:36 +0000 (+0000)
Subject: Bug 519040: After creating a new series, redirect the user to a more useful page... 
X-Git-Tag: bugzilla-3.5.1~41
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1c431100cc82989b40652706add9f024de30ca56;p=thirdparty%2Fbugzilla.git

Bug 519040: After creating a new series, redirect the user to a more useful page - Patch by Frédéric Buclin <LpSolit@gmail.com> r=gerv a=LpSolit
---

diff --git a/chart.cgi b/chart.cgi
index 60a3472539..61bde60eb4 100755
--- a/chart.cgi
+++ b/chart.cgi
@@ -142,19 +142,15 @@ elsif ($action eq "create") {
     
     my $series = new Bugzilla::Series($cgi);
 
-    if (!$series->existsInDatabase()) {
-        $series->writeToDatabase();
-        $vars->{'message'} = "series_created";
-    }
-    else {
-        ThrowUserError("series_already_exists", {'series' => $series});
-    }
+    ThrowUserError("series_already_exists", {'series' => $series})
+      if $series->existsInDatabase;
 
+    $series->writeToDatabase();
+    $vars->{'message'} = "series_created";
     $vars->{'series'} = $series;
 
-    print $cgi->header();
-    $template->process("global/message.html.tmpl", $vars)
-      || ThrowTemplateError($template->error());
+    my $chart = new Bugzilla::Chart($cgi);
+    view($chart);
 }
 elsif ($action eq "edit") {
     detaint_natural($series_id) || ThrowCodeError("invalid_series_id");
diff --git a/template/en/default/global/messages.html.tmpl b/template/en/default/global/messages.html.tmpl
index 766e391138..5c5002fc7b 100644
--- a/template/en/default/global/messages.html.tmpl
+++ b/template/en/default/global/messages.html.tmpl
@@ -742,9 +742,6 @@
       has been created. Note that you may need to wait up to 
       [%+ series.frequency * 2 %] days before there will be enough data for a
       chart of this series to be produced.
-      <br><br>
-      Go back or 
-      <a href="query.cgi?format=create-series">create another series</a>.
     
   [% ELSIF message_tag == "shutdown" %]
     [% title = "$terms.Bugzilla is Down" %]