Datepicker: Only generate yearshtml once. Fixes #7176 - Datepicker shows only single...

diff --git a/ui/jquery.ui.datepicker.js b/ui/jquery.ui.datepicker.js
index ed02335e5418ef1489ae5a7c9143db60a9e8a073..cc27365d3ed2eab9893a5f26721cefd6f2e22116 100644 (file)
--- a/ui/jquery.ui.datepicker.js
+++ b/ui/jquery.ui.datepicker.js
@@ -1579,40 +1579,42 @@ $.extend(Datepicker.prototype, {
                if (!showMonthAfterYear)
                        html += monthHtml + (secondary || !(changeMonth && changeYear) ? ' ' : '');
                // year selection
-               inst.yearshtml = '';
-               if (secondary || !changeYear)
-                       html += '<span class="ui-datepicker-year">' + drawYear + '</span>';
-               else {
-                       // determine range of years to display
-                       var years = this._get(inst, 'yearRange').split(':');
-                       var thisYear = new Date().getFullYear();
-                       var determineYear = function(value) {
-                               var year = (value.match(/c[+-].*/) ? drawYear + parseInt(value.substring(1), 10) :
-                                       (value.match(/[+-].*/) ? thisYear + parseInt(value, 10) :
-                                       parseInt(value, 10)));
-                               return (isNaN(year) ? thisYear : year);
-                       };
-                       var year = determineYear(years[0]);
-                       var endYear = Math.max(year, determineYear(years[1] || ''));
-                       year = (minDate ? Math.max(year, minDate.getFullYear()) : year);
-                       endYear = (maxDate ? Math.min(endYear, maxDate.getFullYear()) : endYear);
-                       inst.yearshtml += '<select class="ui-datepicker-year" ' +
-                               'onchange="DP_jQuery_' + dpuuid + '.datepicker._selectMonthYear(\'#' + inst.id + '\', this, \'Y\');" ' +
-                               'onclick="DP_jQuery_' + dpuuid + '.datepicker._clickMonthYear(\'#' + inst.id + '\');"' +
-                               '>';
-                       for (; year <= endYear; year++) {
-                               inst.yearshtml += '<option value="' + year + '"' +
-                                       (year == drawYear ? ' selected="selected"' : '') +
-                                       '>' + year + '</option>';
-                       }
-                       inst.yearshtml += '</select>';
-                       //when showing there is no need for later update
-                       if( ! $.browser.mozilla ){
-                               html += inst.yearshtml;
-                               inst.yearshtml = null;
-                       } else {
-                               // will be replaced later with inst.yearshtml
-                               html += '<select class="ui-datepicker-year"><option value="' + drawYear + '" selected="selected">' + drawYear + '</option></select>';
+               if ( !inst.yearshtml ) {
+                       inst.yearshtml = '';
+                       if (secondary || !changeYear)
+                               html += '<span class="ui-datepicker-year">' + drawYear + '</span>';
+                       else {
+                               // determine range of years to display
+                               var years = this._get(inst, 'yearRange').split(':');
+                               var thisYear = new Date().getFullYear();
+                               var determineYear = function(value) {
+                                       var year = (value.match(/c[+-].*/) ? drawYear + parseInt(value.substring(1), 10) :
+                                               (value.match(/[+-].*/) ? thisYear + parseInt(value, 10) :
+                                               parseInt(value, 10)));
+                                       return (isNaN(year) ? thisYear : year);
+                               };
+                               var year = determineYear(years[0]);
+                               var endYear = Math.max(year, determineYear(years[1] || ''));
+                               year = (minDate ? Math.max(year, minDate.getFullYear()) : year);
+                               endYear = (maxDate ? Math.min(endYear, maxDate.getFullYear()) : endYear);
+                               inst.yearshtml += '<select class="ui-datepicker-year" ' +
+                                       'onchange="DP_jQuery_' + dpuuid + '.datepicker._selectMonthYear(\'#' + inst.id + '\', this, \'Y\');" ' +
+                                       'onclick="DP_jQuery_' + dpuuid + '.datepicker._clickMonthYear(\'#' + inst.id + '\');"' +
+                                       '>';
+                               for (; year <= endYear; year++) {
+                                       inst.yearshtml += '<option value="' + year + '"' +
+                                               (year == drawYear ? ' selected="selected"' : '') +
+                                               '>' + year + '</option>';
+                               }
+                               inst.yearshtml += '</select>';
+                               //when showing there is no need for later update
+                               if( ! $.browser.mozilla ){
+                                       html += inst.yearshtml;
+                                       inst.yearshtml = null;
+                               } else {
+                                       // will be replaced later with inst.yearshtml
+                                       html += '<select class="ui-datepicker-year"><option value="' + drawYear + '" selected="selected">' + drawYear + '</option></select>';
+                               }
                        }
                }
                html += this._get(inst, 'yearSuffix');