Fix for #141 (#142)

* Fix for #141

Author: CraigHawker

MFiles.VAF.Extensions.Tests/Configuration/ScheduledExecution/DailyTriggerTests.cs

@@ -30,6 +30,19 @@ TimeZoneInfo timezoneInfo
 			}.GetNextExecution(after, timezoneInfo);
 			Assert.AreEqual(expected?.ToUniversalTime(), execution?.ToUniversalTime());
 		}
+		[TestMethod]
+		public void NullAfterDoesNotThrow
+		(
+		)
+		{
+			var execution = new DailyTrigger()
+			{
+				TriggerTimes = new List<TimeSpan>
+				{
+					new TimeSpan(10, 0, 0)
+				}
+			}.GetNextExecution(null);
+		}
 
 		public static IEnumerable<object[]> GetNextExecutionData()
 		{

MFiles.VAF.Extensions/Configuration/ScheduledExecution/DailyTrigger.cs

@@ -59,8 +59,8 @@ public DailyTrigger()
 			timeZoneInfo = timeZoneInfo ?? TimeZoneInfo.Local;
 
 			// When should we start looking?
-			var before = after.Value;
-			after = (after ?? DateTime.UtcNow).ToUniversalTime();
+			var before = (after ?? DateTime.UtcNow);
+			after = before.ToUniversalTime();
 
 			// Convert the time into the timezone we're after.
 			after = TimeZoneInfo.ConvertTime(after.Value, timeZoneInfo);

MFiles.VAF.Extensions/Configuration/ScheduledExecution/Schedule.cs

@@ -108,7 +108,8 @@ public class Schedule
 			// Get the next execution date from the triggers.
 			var next = this.Triggers?
 				.Select(t => t.GetNextExecution(after, timeZoneInfo))
-				.Where(d => d.HasValue && d.Value.DateTime != DateTime.MinValue)
+				.Where(d => d.HasValue)
+				.Where(d => d.Value.DateTime != DateTime.MinValue)
 				.OrderBy(d => d);
 			return next.Any() ? next.First() : null;
 		}