Datepicker: Handle changes to the disabled option. Fixes #8883 - Date…

…picker: Changing disabled option doesn't work.
jquery · Dec 5, 2012 · 598cdae · 598cdae
1 parent 3c2acc3
commit 598cdae
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 0 deletions.
diff --git a/tests/unit/datepicker/datepicker_options.js b/tests/unit/datepicker/datepicker_options.js
@@ -48,6 +48,23 @@ test('option', function() {
 		'Get default settings');
 });
 
+test( "disabled", function() {
+	expect(8);
+	var inp = TestHelpers.datepicker.init('#inp');
+	ok(!inp.datepicker('isDisabled'), 'Initially marked as enabled');
+	ok(!inp[0].disabled, 'Field initially enabled');
+	inp.datepicker('option', 'disabled', true);
+	ok(inp.datepicker('isDisabled'), 'Marked as disabled');
+	ok(inp[0].disabled, 'Field now disabled');
+	inp.datepicker('option', 'disabled', false);
+	ok(!inp.datepicker('isDisabled'), 'Marked as enabled');
+	ok(!inp[0].disabled, 'Field now enabled');
+	inp.datepicker('destroy');
+
+	inp = TestHelpers.datepicker.init('#inp', { disabled: true });
+	ok(inp.datepicker('isDisabled'), 'Initially marked as disabled');
+	ok(inp[0].disabled, 'Field initially disabled');
+})
 test('change', function() {
 	expect( 12 );
 	var inp = TestHelpers.datepicker.init('#inp'),

diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js
@@ -497,6 +497,13 @@ $.extend(Datepicker.prototype, {
 			if (maxDate !== null && settings.dateFormat !== undefined && settings.maxDate === undefined) {
 				inst.settings.maxDate = this._formatDate(inst, maxDate);
 			}
+			if ( "disabled" in settings ) {
+				if ( settings.disabled ) {
+					this._disableDatepicker(target);
+				} else {
+					this._enableDatepicker(target);
+				}
+			}
 			this._attachments($(target), inst);
 			this._autoSize(inst);
 			this._setDate(inst, date);