r3258 committed - dialog: fixed enable and disable methods

r3258 committed - dialog: fixed enable and disable methods


Revision: 3258
Author: rdworth
Date: Sat Sep 19 02:35:53 2009
Log: dialog: fixed enable and disable methods
http://code.google.com/p/jquery-ui/source/detail?r=3258
Modified:
/trunk/tests/unit/dialog/dialog_methods.js
/trunk/ui/jquery.ui.dialog.js
=======================================
--- /trunk/tests/unit/dialog/dialog_methods.js    Tue Sep 15 08:42:41 2009
+++ /trunk/tests/unit/dialog/dialog_methods.js    Sat Sep 19 02:35:53 2009
@@ -68,14 +68,22 @@
    var expected = $('<div></div>').dialog(),
        actual = expected.dialog('enable');
    equals(actual, expected, 'enable is chainable');
-    ok(false, 'missing test - untested code is broken code');
+
+    el = $('<div></div>').dialog({ disabled: true });
+    el.dialog('enable');
+    equals(el.dialog('option', 'disabled'), false, 'enable method sets
disabled option to false');
+    ok(!el.parents('.ui-dialog').hasClass('ui-dialog-disabled'), 'enable
method removes ui-dialog-disabled class from ui-dialog element');
});
test("disable", function() {
    var expected = $('<div></div>').dialog(),
        actual = expected.dialog('disable');
    equals(actual, expected, 'disable is chainable');
-    ok(false, 'missing test - untested code is broken code');
+
+    el = $('<div></div>').dialog({ disabled: false });
+    el.dialog('disable');
+    equals(el.dialog('option', 'disabled'), true, 'disable method sets
disabled option to true');
+    ok(el.parents('.ui-dialog').hasClass('ui-dialog-disabled'), 'disable
method adds ui-dialog-disabled class to ui-dialog element');
});
test("close", function() {
=======================================
--- /trunk/ui/jquery.ui.dialog.js    Thu Sep 17 07:56:03 2009
+++ /trunk/ui/jquery.ui.dialog.js    Sat Sep 19 02:35:53 2009
@@ -442,6 +442,11 @@
                    .removeClass(self.options.dialogClass)
                    .addClass(uiDialogClasses + value);
                break;
+            case "disabled":
+                (value
+                    ? uiDialog.addClass('ui-dialog-disabled')
+                    : uiDialog.removeClass('ui-dialog-disabled'));
+                break;
            case "draggable":
                (value
                    ? self._makeDraggable()