Refactors _updateAlternate() method

benjamin-albert · benjamin-albert · commit 15fcf0afb575 · 2015-11-13T15:12:40.000+02:00
As far as i'm aware .each(function() { $(this).val( ... ) }); is not required to update the value of all elements in the set of matched element.
diff --git a/ui/widgets/datepicker.js b/ui/widgets/datepicker.js
@@ -1089,7 +1089,7 @@ $.extend( Datepicker.prototype, {
 			altFormat = this._get( inst, "altFormat" ) || this._get( inst, "dateFormat" );
 			date = this._getDate( inst );
 			dateStr = this.formatDate( altFormat, date, this._getFormatConfig( inst ) );
-			$( altField ).each( function() { $( this ).val( dateStr ); } );
+			$( altField ).val( dateStr );
 		}
 	},
 

Original file line number	Diff line number	Diff line change
`@@ -1089,7 +1089,7 @@ $.extend( Datepicker.prototype, {`
`1089`	`1089`	`altFormat = this._get( inst, "altFormat" ) \|\| this._get( inst, "dateFormat" );`
`1090`	`1090`	`date = this._getDate( inst );`
`1091`	`1091`	`dateStr = this.formatDate( altFormat, date, this._getFormatConfig( inst ) );`
`1092`		`- $( altField ).each( function() { $( this ).val( dateStr ); } );`
	`1092`	`+ $( altField ).val( dateStr );`
`1093`	`1093`	`}`
`1094`	`1094`	`},`
`1095`	`1095`