Date: months method - Clone and then adjust date to avoid formatting issues and fix failing test.

tjvantoll · tjvantoll · commit d82f4f21d162 · 2013-07-30T08:18:14.000-04:00
diff --git a/datepicker-rewrite/date.js b/datepicker-rewrite/date.js
@@ -140,15 +140,16 @@ $.date = function( datestring, formatstring ) {
 		},
 		// specialized for multi-month template, could be used in general
 		months: function( add ) {
-			var result = [],
-				current = date.getMonth();
-			for ( var i = 0; i < add + 1; i++ ) {
-				result.push( this.clone() );
-				this.adjust( "M", 1 );
+			var clone,
+				result = [ this ];
+
+			for ( var i = 0; i < add; i++ ) {
+				clone = this.clone();
+				clone.adjust( "M", i + 1 );
+				result.push( clone );
 			}
 			result[ 0 ].first = true;
 			result[ result.length - 1 ].last = true;
-			date.setMonth( current );
 			return result;
 		},
 		iso8601Week: function(date) {
diff --git a/datepicker-rewrite/picker.js b/datepicker-rewrite/picker.js
@@ -292,11 +292,13 @@ $.widget( "ui.datepicker", {
 	},
 	_buildMultiplePicker: function() {
 		var html = "",
-			// TODO: All months returned by months() are the same?
+			currentDate = this.date,
 			months = this.date.months( this.options.numberOfMonths - 1 ),
 			i = 0;
 
 		for ( i; i < months.length; i++ ) {
+			this.date = months[ i ];
+
 			html += "<div class='ui-datepicker-group'>" +
 				"<div class='ui-datepicker-header ui-widget-header ui-helper-clearfix ui-corner-all'>";
 			if ( months[i].first ) {
@@ -306,8 +308,6 @@ $.widget( "ui.datepicker", {
 				html += this._buildNextLink();
 			}
 
-			// TODO: Change _buildTitlebar and _buildGrid to handle for multiple months.
-			// Right now they just use this.date.
 			html += this._buildTitlebar();
 			html += "</div>";
 			html += this._buildGrid();
@@ -317,6 +317,7 @@ $.widget( "ui.datepicker", {
 		html += "<div class='ui-datepicker-row-break'></div>";
 		html += this._buildButtons();
 
+		this.date = currentDate;
 		return html;
 	},
 	_buildHeader: function() {
