Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fixing the bug with DailyTimeIntervalTrigger related to Timezones #33

Merged
merged 1 commit into from

2 participants

@yorah

As you can see, the fix is very simple. The bug was caused because we were getting the Date property of a DateTimeOffset, thus losing the timezone.

@lahma lahma merged commit 45d1feb into quartznet:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 14, 2011
  1. @yorah

    DailyTimeIntervalTrigger dit not behave correctly when the server was…

    yorah authored
    … running with a time offset different than UTC+0
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 2 deletions.
  1. +1 −2  src/Quartz/TimeOfDay.cs
View
3  src/Quartz/TimeOfDay.cs
@@ -197,10 +197,9 @@ public override int GetHashCode()
if (dateTime == null)
return null;
- DateTimeOffset cal = dateTime.Value.Date;
+ DateTimeOffset cal = new DateTimeOffset(dateTime.Value.Date, dateTime.Value.Offset);
TimeSpan t = new TimeSpan(0, hour, minute, second);
return cal.Add(t);
-
}
public override string ToString()
Something went wrong with that request. Please try again.