From: Olav Vitters <bugzilla-mozilla@vitters.nl>
Date: Thu, 24 Sep 2015 17:00:53 +0000 (+0000)
Subject: Bug 714525: editmilestones.cgi should use ->set_all
X-Git-Tag: release-5.1.1~161
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ad1dd066b5014a0aa7f67edfddc9ab1a24f0423a;p=thirdparty%2Fbugzilla.git

Bug 714525: editmilestones.cgi should use ->set_all
r=dkl,a=glob
---

diff --git a/editmilestones.cgi b/editmilestones.cgi
index 8052bd1aba..b5188821e2 100755
--- a/editmilestones.cgi
+++ b/editmilestones.cgi
@@ -196,9 +196,11 @@ if ($action eq 'update') {
     my $milestone = Bugzilla::Milestone->check({ product => $product,
                                                  name    => $milestone_old_name });
 
-    $milestone->set_name($milestone_name);
-    $milestone->set_sortkey($sortkey);
-    $milestone->set_is_active($isactive);
+    $milestone->set_all({
+        name      => $milestone_name,
+        sortkey   => $sortkey,
+        is_active => $isactive,
+    });
     my $changes = $milestone->update();
     # Reloading the product since the default milestone name
     # could have been changed.