1 files changed, 12 insertions, 12 deletions

diff --git a/editfields.cgi b/editfields.cgi
index 88961a4e4..acb34e2df 100755
--- a/editfields.cgi
+++ b/editfields.cgi
@@ -61,7 +61,7 @@ elsif ($action eq 'new') {
         custom      => 1,
         buglist     => 1,
         visibility_field_id => scalar $cgi->param('visibility_field_id'),
-        visibility_values => [ $cgi->param('visibility_values') ],
+        visibility_values => [ $cgi->multi_param('visibility_values') ],
         value_field_id => scalar $cgi->param('value_field_id'),
         reverse_desc => scalar $cgi->param('reverse_desc'),
         is_mandatory => scalar $cgi->param('is_mandatory'),
@@ -102,17 +102,17 @@ elsif ($action eq 'update') {
     my $field = new Bugzilla::Field({'name' => $name});
     $field || ThrowUserError('customfield_nonexistent', {'name' => $name});
 
-    $field->set_description($cgi->param('desc'));
-    $field->set_long_desc($cgi->param('long_desc'));
-    $field->set_sortkey($cgi->param('sortkey'));
-    $field->set_in_new_bugmail($cgi->param('new_bugmail'));
-    $field->set_enter_bug($cgi->param('enter_bug'));
-    $field->set_obsolete($cgi->param('obsolete'));
-    $field->set_is_mandatory($cgi->param('is_mandatory'));
-    $field->set_visibility_field($cgi->param('visibility_field_id'));
-    $field->set_visibility_values([ $cgi->param('visibility_values') ]);
-    $field->set_value_field($cgi->param('value_field_id'));
-    $field->set_reverse_desc($cgi->param('reverse_desc'));
+    $field->set_description(scalar $cgi->param('desc'));
+    $field->set_long_desc(scalar $cgi->param('long_desc'));
+    $field->set_sortkey(scalar $cgi->param('sortkey'));
+    $field->set_in_new_bugmail(scalar $cgi->param('new_bugmail'));
+    $field->set_enter_bug(scalar $cgi->param('enter_bug'));
+    $field->set_obsolete(scalar $cgi->param('obsolete'));
+    $field->set_is_mandatory(scalar $cgi->param('is_mandatory'));
+    $field->set_visibility_field(scalar $cgi->param('visibility_field_id'));
+    $field->set_visibility_values([ $cgi->multi_param('visibility_values') ]);
+    $field->set_value_field(scalar $cgi->param('value_field_id'));
+    $field->set_reverse_desc(scalar $cgi->param('reverse_desc'));
     $field->update();
 
     delete_token($token);