Skip to content

Commit 959cb5e

Browse files
author
tristanlatr
committed
Fix resend_emails_after option
1 parent 4bb4c64 commit 959cb5e

File tree

1 file changed

+13
-12
lines changed

1 file changed

+13
-12
lines changed

wpwatcher/email.py

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -170,18 +170,19 @@ def should_notify(
170170
)
171171
should = False
172172

173-
if (
174-
wp_report["last_email"]
175-
and datetime.strptime(wp_report["datetime"], DATE_FORMAT)
176-
- datetime.strptime(wp_report["last_email"], DATE_FORMAT)
177-
< self.resend_emails_after
178-
and (not last_wp_report or last_wp_report["status"] != wp_report["status"])
179-
):
180-
# No report notice
181-
log.info(
182-
f"Not sending WPWatcher {wp_report['status']} email report for site {wp_report['site']} because already sent in the last {self.resend_emails_after}."
183-
)
184-
should = False
173+
if last_wp_report is not None:
174+
if (
175+
last_wp_report["last_email"] is not None
176+
and datetime.strptime(wp_report["datetime"], DATE_FORMAT)
177+
- datetime.strptime(last_wp_report["last_email"], DATE_FORMAT)
178+
< self.resend_emails_after
179+
and last_wp_report["status"] == wp_report["status"]
180+
):
181+
# No report notice
182+
log.info(
183+
f"Not sending WPWatcher {wp_report['status']} email report for site {wp_report['site']} because already sent in the last {self.resend_emails_after}."
184+
)
185+
should = False
185186

186187
return should
187188

0 commit comments

Comments
 (0)