Skip to content

Commit 3e98e73

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 b8e14e7.
1 parent 66d6766 commit 3e98e73

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
@@ -474,16 +474,6 @@ test('setDate', function() {
474474
var dateAndTimeClone = new Date(2008, 3 - 1, 28, 1, 11, 0);
475475
inp.datepicker('setDate', dateAndTimeToSet);
476476
equals(dateAndTimeToSet.getTime(), dateAndTimeClone.getTime(), 'Date object passed should not be changed by setDate');
477-
// Test onSelect callback is executed when using setDate
478-
inp.datepicker('destroy');
479-
var testDate = null;
480-
inp.datepicker({
481-
onSelect: function(dateText, inst) {
482-
testDate = new Date(dateText);
483-
}
484-
});
485-
inp.datepicker('setDate', date2);
486-
equals(date2.getTime(), testDate.getTime(), 'onSelect is called after setDate');
487477
});
488478

489479
test('altField', function() {

ui/jquery.ui.datepicker.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1384,14 +1384,6 @@ $.extend(Datepicker.prototype, {
13841384
if (inst.input) {
13851385
inst.input.val(clear ? '' : this._formatDate(inst));
13861386
}
1387-
1388-
var onSelect = this._get(inst, 'onSelect');
1389-
if (onSelect) {
1390-
var dateStr = this._formatDate(inst);
1391-
1392-
// trigger custom callback
1393-
onSelect.apply((inst.input ? inst.input[0] : null), [dateStr, inst]);
1394-
}
13951387
},
13961388

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

0 commit comments

Comments
 (0)