Preventing setting defaultDate on an empty input to set an (incorrect) input value.

harbulot · harbulot · commit cc80af42fe1c · 2011-03-13T20:27:16.000Z
diff --git a/jquery-ui-timepicker-addon.js b/jquery-ui-timepicker-addon.js
@@ -721,7 +721,7 @@ $.fn.extend({
 	datetimepicker: function(o) {
 		o = o || {};
 		var $input = this,
-			tmp_args = arguments;
+		tmp_args = arguments;
 
 		if (typeof(o) == 'string'){
 			if(o == 'getDate') 
@@ -908,8 +908,8 @@ $.datepicker._setTimeDatepicker = function(target, date, withDate) {
 			}
 			else tp_date = new Date(date.getTime());
 			if (tp_date.toString() == 'Invalid Date') tp_date = undefined;
+			this._setTime(inst, tp_date);
 		}
-		this._setTime(inst, tp_date);
 	}
 
 };

Original file line number	Diff line number	Diff line change
`@@ -721,7 +721,7 @@ $.fn.extend({`
`721`	`721`	`datetimepicker: function(o) {`
`722`	`722`	`o = o \|\| {};`
`723`	`723`	`var $input = this,`
`724`		`- tmp_args = arguments;`
	`724`	`+ tmp_args = arguments;`
`725`	`725`
`726`	`726`	`if (typeof(o) == 'string'){`
`727`	`727`	`if(o == 'getDate')`
`@@ -908,8 +908,8 @@ $.datepicker._setTimeDatepicker = function(target, date, withDate) {`
`908`	`908`	`}`
`909`	`909`	`else tp_date = new Date(date.getTime());`
`910`	`910`	`if (tp_date.toString() == 'Invalid Date') tp_date = undefined;`
	`911`	`+ this._setTime(inst, tp_date);`
`911`	`912`	`}`
`912`		`- this._setTime(inst, tp_date);`
`913`	`913`	`}`
`914`	`914`
`915`	`915`	`};`