From e827a0e280c5652a3f49cea9abf35ccbb9939bda Mon Sep 17 00:00:00 2001 From: Scott González Date: Tue, 9 Aug 2011 08:26:33 -0400 Subject: Revert "Datepicker: Added onSelect.apply() call to _setDate method if onSelect is defined. Fixed #6264 - Datepicker: onSelect does not fire when setDate is called" This reverts commit effdd5d19c534f8445ebafe4212278c4366b0041. --- tests/unit/datepicker/datepicker_options.js | 10 ---------- ui/jquery.ui.datepicker.js | 8 -------- 2 files changed, 18 deletions(-) diff --git a/tests/unit/datepicker/datepicker_options.js b/tests/unit/datepicker/datepicker_options.js index 0a3d5cdac..272af9dac 100644 --- a/tests/unit/datepicker/datepicker_options.js +++ b/tests/unit/datepicker/datepicker_options.js @@ -469,16 +469,6 @@ test('setDate', function() { var dateAndTimeClone = new Date(2008, 3 - 1, 28, 1, 11, 0); inp.datepicker('setDate', dateAndTimeToSet); equals(dateAndTimeToSet.getTime(), dateAndTimeClone.getTime(), 'Date object passed should not be changed by setDate'); - // Test onSelect callback is executed when using setDate - inp.datepicker('destroy'); - var testDate = null; - inp.datepicker({ - onSelect: function(dateText, inst) { - testDate = new Date(dateText); - } - }); - inp.datepicker('setDate', date2); - equals(date2.getTime(), testDate.getTime(), 'onSelect is called after setDate'); }); test('altField', function() { diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js index b125f9457..ade580220 100644 --- a/ui/jquery.ui.datepicker.js +++ b/ui/jquery.ui.datepicker.js @@ -1391,14 +1391,6 @@ $.extend(Datepicker.prototype, { if (inst.input) { inst.input.val(clear ? '' : this._formatDate(inst)); } - - var onSelect = this._get(inst, 'onSelect'); - if (onSelect) { - var dateStr = this._formatDate(inst); - - // trigger custom callback - onSelect.apply((inst.input ? inst.input[0] : null), [dateStr, inst]); - } }, /* Retrieve the date(s) directly. */ -- cgit v1.2.3