diff options
author | Thomas Steur <tsteur@users.noreply.github.com> | 2020-04-09 09:36:29 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-09 09:36:29 +0300 |
commit | 7c52d9df25e7863bde0c92345cc270c1fc6e4f12 (patch) | |
tree | bd1d71d6f7a076b88f24ef79d2bbd0d7a92e6226 | |
parent | fe27dcb12f0d90a951b392321b348c7697d9d781 (diff) |
Make sure to clean up tracking failures before sending email notification (#15798)
Feedback from a customer... Eg the daily `cleanupTrackingFailures()` action might be only executed after the weekly `notifyTrackingFailures` therefore we should try to clean up failures first and then check if any are left. Avoids the case where a user opens hours later the email they receive and then there are no tracking failures reported. This could still happen but it's a bit less likely.
-rw-r--r-- | plugins/CoreAdminHome/Tasks.php | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/plugins/CoreAdminHome/Tasks.php b/plugins/CoreAdminHome/Tasks.php index 1e3933f972..ad81187aa8 100644 --- a/plugins/CoreAdminHome/Tasks.php +++ b/plugins/CoreAdminHome/Tasks.php @@ -195,6 +195,7 @@ class Tasks extends \Piwik\Plugin\Tasks */ public function notifyTrackingFailures() { + $this->cleanupTrackingFailures(); $failures = $this->trackingFailures->getAllFailures(); $general = Config::getInstance()->General; if (!empty($failures) && $general['enable_tracking_failures_notification']) { |