- Autoresponder end_date check fails on disabled autoresponder, Fixes: #3823

Marius Burkard · Marius Burkard · commit 1d46a3ad84e8 · 2016-04-20T16:58:03.000+02:00
diff --git a/interface/lib/classes/validate_autoresponder.inc.php b/interface/lib/classes/validate_autoresponder.inc.php
@@ -60,7 +60,7 @@ function end_date($field_name, $field_value, $validator)
 		$end_date_tstamp = mktime($end_date_array['hour'], $end_date_array['minute'], $end_date_array['second'], $end_date_array['month'], $end_date_array['day'], $end_date_array['year']);
 		
 		// End date has to be > start date
-		if($end_date_tstamp <= $start_date_tstamp) {
+		if($end_date_tstamp <= $start_date_tstamp && ($start_date || $field_value)) {
 			return $app->tform->lng($validator['errmsg']).'<br />';
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ function end_date($field_name, $field_value, $validator)`
`60`	`60`	`$end_date_tstamp = mktime($end_date_array['hour'], $end_date_array['minute'], $end_date_array['second'], $end_date_array['month'], $end_date_array['day'], $end_date_array['year']);`
`61`	`61`
`62`	`62`	`// End date has to be > start date`
`63`		`- if($end_date_tstamp <= $start_date_tstamp) {`
	`63`	`+ if($end_date_tstamp <= $start_date_tstamp && ($start_date \|\| $field_value)) {`
`64`	`64`	`return $app->tform->lng($validator['errmsg']).'<br />';`
`65`	`65`	`}`
`66`	`66`	`}`