From: rojanu <aliustek@gmail.com>
Date: Sat, 15 Oct 2011 13:07:04 +0000 (+0200)
Subject: Bug 686630: editvalues.cgi should use set_all()
X-Git-Tag: bugzilla-4.3.1~226
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ef4e19c923d37a5bb1b2f8fdf5bda3ee9e138b16;p=thirdparty%2Fbugzilla.git

Bug 686630: editvalues.cgi should use set_all()
r/a=mkanat
---

diff --git a/editvalues.cgi b/editvalues.cgi
index 1a8d895311..b06ec63bc9 100755
--- a/editvalues.cgi
+++ b/editvalues.cgi
@@ -181,12 +181,15 @@ if ($action eq 'edit') {
 if ($action eq 'update') {
     check_token_data($token, 'edit_field_value');
     $vars->{'value_old'} = $value->name;
+    my %params = (
+        name    => scalar $cgi->param('value_new'),
+        sortkey => scalar $cgi->param('sortkey'),
+        visibility_value => scalar $cgi->param('visibility_value_id'),
+    );
     if ($cgi->should_set('is_active')) {
-        $value->set_is_active($cgi->param('is_active'));
+        params{is_active} = $cgi->param('is_active');
     }
-    $value->set_name($cgi->param('value_new'));
-    $value->set_sortkey($cgi->param('sortkey'));
-    $value->set_visibility_value($cgi->param('visibility_value_id'));
+    $value->set_all(\%params);
     $vars->{'changes'} = $value->update();
     delete_token($token);
     $vars->{'message'} = 'field_value_updated';