test dirty onchange

instead of blindly setting dirty, check if the filed has a different value than it originally did.

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

2 files changed, 9 insertions, 9 deletions

diff --git a/install/ui/aci.js b/install/ui/aci.js
index 70fe6d925..077cbebdc 100644
--- a/install/ui/aci.js
+++ b/install/ui/aci.js
@@ -234,7 +234,7 @@ IPA.attributes_widget = function(spec) {
                 click: function(){
                     $('.aci-attribute', that.table).
                         attr('checked', $(this).attr('checked'));
-                    that.set_dirty(true);
+                    that.set_dirty(that.test_dirty());
                 }
             })
         })).append($('<th/>', {
@@ -298,7 +298,7 @@ IPA.attributes_widget = function(spec) {
                 value: value,
                 'class': 'aci-attribute',
                 click: function() {
-                    that.set_dirty(true);
+                    that.set_dirty(that.test_dirty());
                 }
             }));
             td =  $('<td/>').appendTo(aci_tr);
@@ -335,7 +335,7 @@ IPA.attributes_widget = function(spec) {
                     value: value,
                     'class': 'aci-attribute',
                     change: function() {
-                        that.set_dirty(true);
+                        that.set_dirty(that.test_dirty());
                     }
                 }));
 
diff --git a/install/ui/widget.js b/install/ui/widget.js
index 14a40a577..80906a2f5 100644
--- a/install/ui/widget.js
+++ b/install/ui/widget.js
@@ -728,7 +728,7 @@ IPA.checkbox_widget = function (spec) {
 
         var input = $('input[name="'+that.name+'"]', that.container);
         input.change(function() {
-            that.set_dirty(true);
+            that.set_dirty(that.test_dirty());
         });
 
         var undo = that.get_undo();
@@ -802,7 +802,7 @@ IPA.checkboxes_widget = function (spec) {
 
         var input = $('input[name="'+that.name+'"]', that.container);
         input.change(function() {
-            that.set_dirty(true);
+            that.set_dirty(that.test_dirty());
         });
 
         var undo = that.get_undo();
@@ -880,7 +880,7 @@ IPA.radio_widget = function(spec) {
 
         var input = $('input[name="'+that.name+'"]', that.container);
         input.change(function() {
-            that.set_dirty(true);
+            that.set_dirty(that.test_dirty());
         });
 
         var undo = that.get_undo();
@@ -957,7 +957,7 @@ IPA.select_widget = function(spec) {
 
         that.select = $('select[name="'+that.name+'"]', that.container);
         that.select.change(function() {
-            that.set_dirty(true);
+            that.set_dirty(that.test_dirty());
         });
 
         var undo = that.get_undo();
@@ -1575,7 +1575,7 @@ IPA.entity_select_widget = function(spec) {
         that.entity_select = $('<select/>', {
             id: that.name + '-entity-select',
             change: function(){
-                that.set_dirty(true);
+                that.set_dirty(that.test_dirty());
             }
         }).appendTo(container);
 
@@ -1586,7 +1586,7 @@ IPA.entity_select_widget = function(spec) {
             style: 'display: none;',
             keyup: function(){
                 populate_select();
-                that.set_dirty(true);
+                that.set_dirty(that.test_dirty());
             }
         }).appendTo(container);