Commit 4f5cad99 authored by erik%dasbistro.com's avatar erik%dasbistro.com

Bug 276660

"1st of the Month" whine generates many many copies of whine mail Patch: erik@dasbistro.com r=justdave, a=justdave
parent d43362f9
...@@ -523,8 +523,9 @@ sub reset_timer { ...@@ -523,8 +523,9 @@ sub reset_timer {
# whine.pl -- there are legitimate circumstances that can cause this, like # whine.pl -- there are legitimate circumstances that can cause this, like
# a set of whines that take a very long time to execute, so it's done # a set of whines that take a very long time to execute, so it's done
# quietly. # quietly.
if (grep(/^$schedule_id$/, @seen_schedules)) { if (grep($_ == $schedule_id, @seen_schedules)) {
null_schedule($schedule_id); null_schedule($schedule_id);
return;
} }
push @seen_schedules, $schedule_id; push @seen_schedules, $schedule_id;
...@@ -652,7 +653,7 @@ sub get_next_date { ...@@ -652,7 +653,7 @@ sub get_next_date {
else { # it's a number, so we set it for that calendar day else { # it's a number, so we set it for that calendar day
$add_days = $day - $now_day; $add_days = $day - $now_day;
# If it's already beyond that day this month, set it to the next one # If it's already beyond that day this month, set it to the next one
if ($add_days < 0) { if ($add_days <= 0) {
$add_days += $daysinmonth[$now_month]; $add_days += $daysinmonth[$now_month];
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment