Skip to content

Commit e827a0e

Browse files
committed
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.
1 parent f8f38e8 commit e827a0e

File tree

2 files changed

+0
-18
lines changed

2 files changed

+0
-18
lines changed

tests/unit/datepicker/datepicker_options.js

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -469,16 +469,6 @@ test('setDate', function() {
469469
var dateAndTimeClone = new Date(2008, 3 - 1, 28, 1, 11, 0);
470470
inp.datepicker('setDate', dateAndTimeToSet);
471471
equals(dateAndTimeToSet.getTime(), dateAndTimeClone.getTime(), 'Date object passed should not be changed by setDate');
472-
// Test onSelect callback is executed when using setDate
473-
inp.datepicker('destroy');
474-
var testDate = null;
475-
inp.datepicker({
476-
onSelect: function(dateText, inst) {
477-
testDate = new Date(dateText);
478-
}
479-
});
480-
inp.datepicker('setDate', date2);
481-
equals(date2.getTime(), testDate.getTime(), 'onSelect is called after setDate');
482472
});
483473

484474
test('altField', function() {

ui/jquery.ui.datepicker.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1391,14 +1391,6 @@ $.extend(Datepicker.prototype, {
13911391
if (inst.input) {
13921392
inst.input.val(clear ? '' : this._formatDate(inst));
13931393
}
1394-
1395-
var onSelect = this._get(inst, 'onSelect');
1396-
if (onSelect) {
1397-
var dateStr = this._formatDate(inst);
1398-
1399-
// trigger custom callback
1400-
onSelect.apply((inst.input ? inst.input[0] : null), [dateStr, inst]);
1401-
}
14021394
},
14031395

14041396
/* Retrieve the date(s) directly. */

0 commit comments

Comments
 (0)