From ef4e19c923d37a5bb1b2f8fdf5bda3ee9e138b16 Mon Sep 17 00:00:00 2001
From: rojanu <aliustek@gmail.com>
Date: Sat, 15 Oct 2011 15:07:04 +0200
Subject: Bug 686630: editvalues.cgi should use set_all() r/a=mkanat

---
 editvalues.cgi | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

(limited to 'editvalues.cgi')

diff --git a/editvalues.cgi b/editvalues.cgi
index 1a8d89531..b06ec63bc 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';
-- 
cgit v1.2.3-24-g4f1b