New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix potential time shift in EveryJob #181

Merged
merged 1 commit into from Nov 28, 2015
Jump to file or symbol
Failed to load files and symbols.
+1 −2
Diff settings

Always

Just for now

fix potential time shift in EveryJob. We use the last @next_time valu…

…e and add @frequency instead of relying of the trigger time which may vary
  • Loading branch information...
Korrigan committed Nov 27, 2015
commit eeaba4505d32906d69e819e48c41c83077dfefa0
@@ -545,7 +545,7 @@ def set_next_time(trigger_time, is_post=false)
@next_time =
if @first_at == nil || @first_at < Time.now
(trigger_time || Time.now) + @frequency
(@next_time || trigger_time || Time.now) + @frequency
else
@first_at
end
@@ -637,4 +637,3 @@ def next_time_from(time)
end
end
end
ProTip! Use n and p to navigate between commits in a pull request.