From b7179b75a8001713fb1f69f94c9e3440848f0839 Mon Sep 17 00:00:00 2001
From: Marc <sunpack@web.de>
Date: Thu, 30 Apr 2015 10:06:03 +0200
Subject: [PATCH] Some bugfixing, optimization and cleaning

---
 .../static/js/customControl.js                | 75 +++++++++++--------
 .../templates/customControl_settings.jinja2   |  2 +-
 2 files changed, 46 insertions(+), 31 deletions(-)

diff --git a/octoprint_customControl/static/js/customControl.js b/octoprint_customControl/static/js/customControl.js
index a546db0..9d59489 100644
--- a/octoprint_customControl/static/js/customControl.js
+++ b/octoprint_customControl/static/js/customControl.js
@@ -88,10 +88,10 @@
                 var input = {
                     name: ko.observable(element.name),
                     parameter: ko.observable(element.parameter),
-                    default: ko.observable(element.default != "" ? element.default : undefined)
+                    default: ko.observable(element.hasOwnProperty("default") ? element.default : undefined)
                 }
 
-                if (element.hasOwnProperty("slider") && typeof element.slider == "object") {
+                if (element.hasOwnProperty("slider") && _.isObject(element.slider)) {
                     input.slider = {
                         min: ko.observable(element.slider.min),
                         max: ko.observable(element.slider.max),
@@ -139,8 +139,7 @@
                 control.template = ko.observable(control.template);
                 control.regex = ko.observable(control.regex);
                 control.default = ko.observable(control.default || "");
-                control.value = control.default;
-
+                control.value = ko.computed(function () { return control.default(); });
 
                 delete control.key;
                 delete control.template_key;
@@ -284,7 +283,7 @@
             if (element.hasOwnProperty("template")) {
                 data.template = element.template();
                 data.regex = element.regex();
-                data.default = element.default() || "";
+                data.defaultValue = element.default() || "";
 
                 title = "Edit Output";
                 type = "output";
@@ -327,10 +326,10 @@
             {
                 data.input = [];
                 _.each(element.input(), function (element, index, list) {
+                    data.input[index] = ko.mapping.toJS(element);
                     if (element.hasOwnProperty("default")) {
                         data.input[index].defaultValue = element.default;
                     }
-                    data.input[index] = ko.mapping.toJS(element);
                 });
             }
 
@@ -394,9 +393,9 @@
                                 element.regex = ko.observable(ret.regex);
 
                             if (element.hasOwnProperty("default"))
-                                element.default(ret.defaultValue);
+                                element.default(ret.default);
                             else
-                                element.default = ko.observable(ret.defaultValue);
+                                element.default = ko.observable(ret.default);
                         }
                         else
                         {
@@ -490,34 +489,50 @@
         }
        
         self.recursiveDeleteProperties = function (list) {
-            for (var i = 0; i < list.length; i++) {
-                if (!list[i].parent || (list[i].parent.hasOwnProperty("layout") && list[i].parent.layout() != "horizontal_grid"))
-                {
-                    delete list[i].width;
-                    delete list[i].offset;
+            _.each(list, function (element, index, ll) {
+                if (!element.parent || (element.parent.hasOwnProperty("layout") && element.parent.layout() != "horizontal_grid")) {
+                    delete element.width;
+                    delete element.offset;
                 }
 
-                delete list[i].id;
-                delete list[i].parent;
-                delete list[i].processed;
-                delete list[i].output;
-                delete list[i].key;
-                delete list[i].template_key;
-                delete list[i].value;
+                if (element.default == "")
+                    delete element.default;
 
-                if (list[i].hasOwnProperty("width") && list[i].width() == "")
-                    delete list[i].width;
-                if (list[i].hasOwnProperty("offset") && list[i].offset() == "")
-                    delete list[i].offset;
+                delete element.id;
+                delete element.parent;
+                delete element.processed;
+                delete element.output;
+                delete element.key;
+                delete element.template_key;
+                delete element.value;
 
-                if (!list[i].hasOwnProperty("name") || list[i].name() == "") {
-                    delete list[i].name;
-                    delete list[i].collapsable;
+                if (element.hasOwnProperty("input")) {
+                    _.each(element.input(), function (e, i, l) {
+                        if (e.default == "")
+                            delete e.default;
+
+                        delete e.value;
+                    });
                 }
 
-                if (list[i].hasOwnProperty("children"))
-                     self.recursiveDeleteProperties(list[i].children());
-            }
+                if (element.hasOwnProperty("width") && element.width() == "")
+                    delete element.width;
+                if (element.hasOwnProperty("offset") && element.offset() == "")
+                    delete element.offset;
+
+                if (!element.hasOwnProperty("name") || element.name() == "") {
+                    delete element.name;
+                    delete element.collapsed;
+                }
+
+
+                if (element.hasOwnProperty("children")) {
+                    if (element.hasOwnProperty("collapsed") && !element.collapsed())
+                        delete element.collapsed;
+
+                    self.recursiveDeleteProperties(element.children());
+                }
+            });
         }
         self.onSettingsBeforeSave = function () {
             self.recursiveDeleteProperties(self.controlsFromServer);
diff --git a/octoprint_customControl/templates/customControl_settings.jinja2 b/octoprint_customControl/templates/customControl_settings.jinja2
index f336a5e..d59e31b 100644
--- a/octoprint_customControl/templates/customControl_settings.jinja2
+++ b/octoprint_customControl/templates/customControl_settings.jinja2
@@ -78,7 +78,7 @@
     <form class="form-inline custom_control" style="min-height:15px; border:1px dotted #000000" data-bind="contextMenu: { menuSelector: '#commandContextMenu', menuSelected: $root.controlContextMenu }, attr: { 'id': id }">
         <!-- ko template: { name: 'settingsCustomControls_controlTemplate_input', data: $data, if: $data.hasOwnProperty('input') } --><!-- /ko -->
         <!-- ko template: { name: 'settingsCustomControls_controlTemplate_command', data: $data, if: $data.hasOwnProperty('command') || $data.hasOwnProperty('commands') || $data.hasOwnProperty('script') || $data.hasOwnProperty('javascript') } --><!-- /ko -->
-        <!-- ko template: { name: 'settingsCustomControls_controlTemplate_output', data: $data, if: $data.hasOwnProperty('defaultValue') && defaultValue() } --><!-- /ko -->
+        <!-- ko template: { name: 'settingsCustomControls_controlTemplate_output', data: $data, if: $data.hasOwnProperty('value') } --><!-- /ko -->
     </form>
 </script>
 <script type="text/html" id="settingsCustomControls_controlTemplate_input">