make sure that values are written when they change (closes #50)

AIFDR · Jun 21, 2011 · 8af7b51 · 8af7b51
1 parent 53a101b
commit 8af7b51
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/app/static/script/app/TsuDat2.js b/app/static/script/app/TsuDat2.js
@@ -82,9 +82,6 @@ var TsuDat2 = Ext.extend(gxp.Viewer, {
                             }
                         }]
                     }
-                    /*handler: function() {
-                        window.location.href = "/";
-                    }*/
                 }, "-", "-", "->", {
                     xtype: "tbtext",
                     ref: "loginName",

diff --git a/app/static/script/app/TsuDat2/SimulationParameters.js b/app/static/script/app/TsuDat2/SimulationParameters.js
@@ -18,6 +18,7 @@ TsuDat2.SimulationParameters = Ext.extend(gxp.plugins.WizardStep, {
     ptype: "app_simulationparameters",
 
     addOutput: function(config) {
+        var data; // result data
         var output = TsuDat2.SimulationParameters.superclass.addOutput.call(this, {
             xtype: "form",
             labelWidth: 80,
@@ -105,10 +106,14 @@ TsuDat2.SimulationParameters = Ext.extend(gxp.plugins.WizardStep, {
             }],
             listeners: {
                 "clientvalidation": function(fp, valid) {
-                    if (valid !== this.valid) {
-                        var data = output.getForm().getFieldValues();
-                        data.model_setup = data.model_setup.value;
-                        this.setValid(valid, data);
+                    if (valid) {
+                        values = output.getForm().getFieldValues();
+                        values.model_setup = values.model_setup.value;
+                        var newData = Ext.encode(values);
+                        if (data !== newData) {
+                            data = newData;
+                            this.setValid(valid, data);
+                        }
                     }
                 },
                 scope: this