Skip to content
Permalink
Browse files

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 effdd5d.
  • Loading branch information...
scottgonzalez committed Aug 9, 2011
1 parent f8f38e8 commit e827a0e280c5652a3f49cea9abf35ccbb9939bda
Showing with 0 additions and 18 deletions.
  1. +0 −10 tests/unit/datepicker/datepicker_options.js
  2. +0 −8 ui/jquery.ui.datepicker.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() {
@@ -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. */

0 comments on commit e827a0e

Please sign in to comment.
You can’t perform that action at this time.