Removed CSV creation from UI

Creating CSV values in UI is unnecessary and error-prone because server converts them back to list. Possible problems with values containing commas may occur.  All occurrences of CSV joining were therefore removed.

https://fedorahosted.org/freeipa/ticket/2227

1 files changed, 5 insertions, 7 deletions

diff --git a/install/ui/add.js b/install/ui/add.js
index 9b473ccc2..671d3f1fc 100644
--- a/install/ui/add.js
+++ b/install/ui/add.js
@@ -123,16 +123,14 @@ IPA.entity_adder_dialog = function(spec) {
             var field = fields[j];
 
             var values = record[field.param];
-            if (!values) continue;
-
-            // TODO: Handle multi-valued attributes like in detail facet's update()
-            var value = values.join(',');
-            if (!value) continue;
+            if (!values || values.length === 0) continue;
 
             if (field.param === pkey_name) {
-                command.add_arg(value);
+                command.add_arg(values[0]);
+            } else if (values.length === 1) {
+                command.set_option(field.param, values[0]);
             } else {
-                command.set_option(field.param, value);
+                command.set_option(field.param, values);
             }
         }