Skip to content
Permalink
Browse files

Datepicker: Use 'that' instead of 'self'. Partial fix for #5404 - rem…

…ove uses of 'var self = this;'
  • Loading branch information
scottgonzalez committed Feb 12, 2012
1 parent 41a1472 commit d2d301f074f814871a68e4c386bf619908f188c0
Showing with 3 additions and 4 deletions.
  1. +3 −4 ui/jquery.ui.datepicker.js
@@ -711,8 +711,7 @@ $.extend(Datepicker.prototype, {

/* Generate the date picker content. */
_updateDatepicker: function(inst) {
var self = this;
self.maxRows = 4; //Reset the max number of rows being displayed (see #7043)
this.maxRows = 4; //Reset the max number of rows being displayed (see #7043)
var borders = $.datepicker._getBorders(inst.dpDiv);
instActive = inst; // for delegate hover events
inst.dpDiv.empty().append(this._generateHTML(inst));
@@ -802,10 +801,10 @@ $.extend(Datepicker.prototype, {
if (this._datepickerShowing) {
var showAnim = this._get(inst, 'showAnim');
var duration = this._get(inst, 'duration');
var self = this;
var that = this;
var postProcess = function() {
$.datepicker._tidyDialog(inst);
self._curInst = null;
that._curInst = null;
};

// DEPRECATED: after BC for 1.8.x $.effects[ showAnim ] is not needed

0 comments on commit d2d301f

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