Fix #840: Fix GetOccurrences to properly deal with periodStart's TZ

minichma · minichma · commit d20cf384fc04 · 2025-07-15T15:19:54.000+02:00
diff --git a/Ical.Net/Evaluation/RecurrenceUtil.cs b/Ical.Net/Evaluation/RecurrenceUtil.cs
@@ -22,15 +22,8 @@ public static IEnumerable<Occurrence> GetOccurrences(IRecurrable recurrable, Cal
             return [];
         }
 
-        // Ensure the start time is associated with the object being queried
         var start = recurrable.Start;
 
-        // Change the time zone of periodStart as needed
-        // so they can be used during the evaluation process.
-
-        if (periodStart != null)
-            periodStart = new CalDateTime(periodStart.Date, periodStart.Time, start.TzId);
-
         var periods = evaluator.Evaluate(start, periodStart, options);
         if (periodStart != null)
         {

Original file line number	Diff line number	Diff line change
`@@ -22,15 +22,8 @@ public static IEnumerable<Occurrence> GetOccurrences(IRecurrable recurrable, Cal`
`22`	`22`	`return [];`
`23`	`23`	`}`
`24`	`24`
`25`		`- // Ensure the start time is associated with the object being queried`
`26`	`25`	`var start = recurrable.Start;`
`27`	`26`
`28`		`- // Change the time zone of periodStart as needed`
`29`		`- // so they can be used during the evaluation process.`
`30`		`-`
`31`		`- if (periodStart != null)`
`32`		`- periodStart = new CalDateTime(periodStart.Date, periodStart.Time, start.TzId);`
`33`		`-`
`34`	`27`	`var periods = evaluator.Evaluate(start, periodStart, options);`
`35`	`28`	`if (periodStart != null)`
`36`	`29`	`{`