From dbeae4e3f43a15c84d95447c0c38266f06402ea2 Mon Sep 17 00:00:00 2001
From: Adam Young <ayoung@redhat.com>
Date: Thu, 16 Jun 2011 15:37:27 -0400
Subject: 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
---
 install/ui/aci.js    |  6 +++---
 install/ui/widget.js | 12 ++++++------
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/install/ui/aci.js b/install/ui/aci.js
index 70fe6d92..077cbebd 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 14a40a57..80906a2f 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);
 
-- 
cgit