From 8b041cd1a359b0a3496c16abbc6a32c804084d83 Mon Sep 17 00:00:00 2001
From: "Endi S. Dewata" <edewata@redhat.com>
Date: Fri, 4 Mar 2011 15:05:02 -0600
Subject: Fixed memory leak caused by is_dirty dialogs.

Ticket 1054
---
 install/ui/ipa.js | 26 ++++++++++++++------------
 1 file changed, 14 insertions(+), 12 deletions(-)

(limited to 'install/ui/ipa.js')

diff --git a/install/ui/ipa.js b/install/ui/ipa.js
index ec15332f..cbe009b9 100644
--- a/install/ui/ipa.js
+++ b/install/ui/ipa.js
@@ -135,24 +135,26 @@ var IPA = ( function () {
         if (IPA.current_entity){
             var facet_name =   IPA.current_facet(IPA.current_entity);
             var facet = IPA.current_entity.facets_by_name[facet_name];
+
             if (facet.is_dirty()){
-                var message_box =  $("<div/>",{
-                    html: IPA.messages.dialogs.dirty_message
-                }).
-                    appendTo($("#navigation"));
 
-                var buttons = {};
+                var dialog = IPA.dialog({
+                    title: IPA.messages.dialogs.dirty_title,
+                    width: '20em'
+                });
 
-                buttons[IPA.messages.buttons.ok] = function() {
-                    $(this).dialog("close");
+                dialog.create = function() {
+                    dialog.container.append(IPA.messages.dialogs.dirty_message);
                 };
 
-                message_box.dialog({
-                    title: IPA.messages.dialogs.dirty_title,
-                    modal:true,
-                    width: '20em',
-                    buttons: buttons
+                dialog.add_button(IPA.messages.buttons.ok, function() {
+                    dialog.close();
                 });
+
+                dialog.init();
+
+                dialog.open($('#navigation'));
+
                 return false;
             }
         }
-- 
cgit