From 697af3e1f8b4102df2bde5c7d04f8e7d4dad4369 Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Wed, 23 Feb 2011 12:35:45 -0600
Subject: Save changes before modifying association.

In a details page, usually any changes done to the fields will not be
applied until the user clicks the Update button. However, if the page
contains an association table, any addition/deletion to the table will
be applied immediately.

To avoid any confusion, the user is now required to save or reset all
changes to the page before modifying the association. A dialog box will
appear if the page contains any unsaved changes.
---
 install/ui/associate.js                 | 42 ++++++++++++++++++++++++++++++---
 install/ui/ipa.js                       |  4 ++--
 install/ui/test/data/i18n_messages.json |  3 ++-
 install/ui/test/data/ipa_init.json      |  3 ++-
 4 files changed, 45 insertions(+), 7 deletions(-)

(limited to 'install/ui')

diff --git a/install/ui/associate.js b/install/ui/associate.js
index 600cd55c3..2f1a28d5a 100644
--- a/install/ui/associate.js
+++ b/install/ui/associate.js
@@ -351,6 +351,28 @@ IPA.association_table_widget = function (spec) {
 
         that.table_setup(container);
 
+        var dialog = $('<div/>', {
+            html: IPA.messages.dialogs.dirty_message
+        }).appendTo(container);
+
+        var buttons = {};
+
+        buttons[IPA.messages.buttons.ok] = function() {
+            dialog.dialog('close');
+        };
+
+        dialog.dialog({
+            autoOpen: false,
+            title: IPA.messages.dialogs.dirty_title,
+            modal: true,
+            width: '20em',
+            buttons: buttons
+        });
+
+        var entity = IPA.get_entity(that.entity_name);
+        var facet_name = IPA.current_facet(entity);
+        var facet = entity.get_facet(facet_name);
+
         var button = $('input[name=remove]', container);
         button.replaceWith(IPA.action_button({
             'label': button.val(),
@@ -359,7 +381,13 @@ IPA.association_table_widget = function (spec) {
                 if ($(this).hasClass('action-button-disabled')) {
                     return false;
                 }
-                that.show_remove_dialog();
+
+                if (facet.is_dirty()) {
+                    dialog.dialog('open');
+                } else {
+                    that.show_remove_dialog();
+                }
+
                 return false;
             }
         }));
@@ -369,8 +397,16 @@ IPA.association_table_widget = function (spec) {
             'label': button.val(),
             'icon': 'ui-icon-plus',
             'click': function() {
-                if ($(this).hasClass('action-button-disabled')) return false;
-                that.show_add_dialog();
+                if ($(this).hasClass('action-button-disabled')) {
+                    return false;
+                }
+
+                if (facet.is_dirty()) {
+                    dialog.dialog('open');
+                } else {
+                    that.show_add_dialog();
+                }
+
                 return false;
             }
         }));
diff --git a/install/ui/ipa.js b/install/ui/ipa.js
index 2c07d742a..ec15332f4 100644
--- a/install/ui/ipa.js
+++ b/install/ui/ipa.js
@@ -137,7 +137,7 @@ var IPA = ( function () {
             var facet = IPA.current_entity.facets_by_name[facet_name];
             if (facet.is_dirty()){
                 var message_box =  $("<div/>",{
-                    html: IPA.messages.dirty
+                    html: IPA.messages.dialogs.dirty_message
                 }).
                     appendTo($("#navigation"));
 
@@ -148,7 +148,7 @@ var IPA = ( function () {
                 };
 
                 message_box.dialog({
-                    title: 'Dirty',
+                    title: IPA.messages.dialogs.dirty_title,
                     modal:true,
                     width: '20em',
                     buttons: buttons
diff --git a/install/ui/test/data/i18n_messages.json b/install/ui/test/data/i18n_messages.json
index 11b02c846..cca68b945 100644
--- a/install/ui/test/data/i18n_messages.json
+++ b/install/ui/test/data/i18n_messages.json
@@ -42,12 +42,13 @@
             },
             "dialogs": {
                 "available": "Available",
+                "dirty_message": "This page has unsaved changes. Please save or revert.",
+                "dirty_title": "Dirty",
                 "hide_already_enrolled": "Hide already enrolled.",
                 "prospective": "Prospective",
                 "remove_empty": "Select ${entity} to be removed.",
                 "remove_title": "Remove ${entity}."
             },
-            "dirty": "This page has unsaved changes. Please save or revert.",
             "facets": {
                 "details": "Settings",
                 "search": "Search"
diff --git a/install/ui/test/data/ipa_init.json b/install/ui/test/data/ipa_init.json
index 99f6a6060..b819dab46 100644
--- a/install/ui/test/data/ipa_init.json
+++ b/install/ui/test/data/ipa_init.json
@@ -9505,12 +9505,13 @@
                     },
                     "dialogs": {
                         "available": "Available",
+                        "dirty_message": "This page has unsaved changes. Please save or revert.",
+                        "dirty_title": "Dirty",
                         "hide_already_enrolled": "Hide already enrolled.",
                         "prospective": "Prospective",
                         "remove_empty": "Select ${entity} to be removed.",
                         "remove_title": "Remove ${entity}."
                     },
-                    "dirty": "This page has unsaved changes. Please save or revert.",
                     "facets": {
                         "details": "Settings",
                         "search": "Search"
-- 
cgit