From: erik%dasbistro.com <>
Date: Tue, 11 Jan 2005 03:31:50 +0000 (+0000)
Subject: Bug 276660
X-Git-Tag: bugzilla-2.19.2~12
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=4f5cad994b3786713c79b831e3af48ea7fe3a9b0;p=thirdparty%2Fbugzilla.git

Bug 276660
"1st of the Month" whine generates many many copies of whine mail
Patch: erik@dasbistro.com
r=justdave, a=justdave
---

diff --git a/whine.pl b/whine.pl
index 25a422a710..920b27d058 100755
--- a/whine.pl
+++ b/whine.pl
@@ -523,8 +523,9 @@ sub reset_timer {
     # 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
     # quietly.
-    if (grep(/^$schedule_id$/, @seen_schedules)) {
+    if (grep($_ == $schedule_id, @seen_schedules)) {
         null_schedule($schedule_id);
+        return;
     }
     push @seen_schedules, $schedule_id;
 
@@ -652,7 +653,7 @@ sub get_next_date {
     else { # it's a number, so we set it for that calendar day
         $add_days = $day - $now_day;
         # 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];
         }
     }