Skip to content

Commit a53fdaa

Browse files
committed
GD-18 sending failed requirements once a day
1 parent 3f3f60d commit a53fdaa

File tree

1 file changed

+17
-6
lines changed

1 file changed

+17
-6
lines changed

os2web_logging.module

Lines changed: 17 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -96,21 +96,32 @@ function os2web_logging_cron() {
9696
$last_cleanup_run = DateTimePlus::createFromTimestamp($last_cleanup_run_ts);
9797
$now = DateTimePlus::createFromTimestamp(time());
9898

99-
$diff = $last_cleanup_run->diff($now);
99+
$cleanupDiff = $last_cleanup_run->diff($now);
100100

101101
// Proceed if the last cleanup happened more than a day ago.
102-
if ($diff->days > 0) {
102+
if ($cleanupDiff->days > 0) {
103103
// Do a cleanup.
104104
_os2web_logging_dblogs_cleanup();
105105

106106
\Drupal::state()
107107
->set('os2web_logging.last_cleanup_run', time());
108108
}
109109

110-
// Checking requirements.
111-
$requirements = LoggingController::getCheckedRequirements();
112-
if (!empty($requirements['error'])) {
113-
_os2web_logging_send_status_failed_email($requirements['error']);
110+
$last_requirements_check_run_ts = \Drupal::state()
111+
->get('os2web_logging.last_requirements_check_run', 0);
112+
$last_requirements_check_run = DateTimePlus::createFromTimestamp($last_requirements_check_run_ts);
113+
114+
$requirementsCheckDiff = $last_requirements_check_run->diff($now);
115+
116+
// Proceed if the last requirements check happened more than a day ago.
117+
if ($requirementsCheckDiff->days > 0) {
118+
// Checking requirements.
119+
$requirements = LoggingController::getCheckedRequirements();
120+
if (!empty($requirements['error'])) {
121+
_os2web_logging_send_status_failed_email($requirements['error']);
122+
\Drupal::state()
123+
->set('os2web_logging.last_requirements_check_run', time());
124+
}
114125
}
115126
}
116127

0 commit comments

Comments
 (0)