Removing sections as special type of object

Sections are changed into pure widget objects. Introduced IPA.composite_widget, basic widget for widget nesting (it's using IPA.widget_container). It's base class for section widgets.

TODO: change old custom sections into custom fields and widgets.

Note: usage of section in HBAC and SUDO is kept - whole logic will be removed in #1515 patch.

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

1 files changed, 15 insertions, 22 deletions

diff --git a/install/ui/add.js b/install/ui/add.js
index c9d5e46b..032302e3 100644
--- a/install/ui/add.js
+++ b/install/ui/add.js
@@ -127,31 +127,24 @@ IPA.entity_adder_dialog = function(spec) {
         var record = {};
         that.save(record);
 
-        var sections = that.sections.values;
-        for (var i=0; i<sections.length; i++) {
-            var section = sections[i];
-
-            var section_fields = section.fields.values;
-            for (var j=0; j<section_fields.length; j++) {
-                var field = section_fields[j];
-
-                var values = record[field.name];
-                if (!values) continue;
-
-                // TODO: Handle multi-valued attributes like in detail facet's update()
-                var value = values.join(',');
-                if (!value) continue;
-
-                if (field.name == pkey_name) {
-                    command.add_arg(value);
-                } else {
-                    command.set_option(field.name, value);
-                }
+        var fields = that.fields.get_fields();
+        for (var j=0; j<fields.length; j++) {
+            var field = fields[j];
+
+            var values = record[field.name];
+            if (!values) continue;
+
+            // TODO: Handle multi-valued attributes like in detail facet's update()
+            var value = values.join(',');
+            if (!value) continue;
+
+            if (field.name == pkey_name) {
+                command.add_arg(value);
+            } else {
+                command.set_option(field.name, value);
             }
         }
 
-        //alert(JSON.stringify(command.to_json()));
-
         if (that.pre_execute_hook) {
             that.pre_execute_hook(command);
         }