[Legacy] Update AlertSchedulers alert creation

Do not create new alerts for already dismissed reminders
This commit is contained in:
Clemente Raposo 2023-02-20 18:38:41 +00:00
parent 452c52ec95
commit 32f352f63c
2 changed files with 6 additions and 3 deletions

View file

@ -52,7 +52,6 @@ class AddScheduledAlerts
*/
public function run(): void
{
$this->addActivitiesAlertsService->run();
$this->addScheduledReminderAlerts->run();
}
}

View file

@ -163,8 +163,12 @@ class AddScheduledReminderAlerts
{
$popupReminders = BeanFactory::getBean('Reminders')->get_full_list(
'',
"reminders.popup = 1 AND (reminders.date_willexecute = -1 OR reminders.date_willexecute BETWEEN "
. $dateTimeNowStamp . " AND " . $dateTimeMaxStamp . ")"
"reminders.popup = 1 AND
reminders.popup_viewed = 0 AND
(
reminders.date_willexecute = -1 OR
reminders.date_willexecute BETWEEN " . $dateTimeNowStamp . " AND " . $dateTimeMaxStamp . "
)"
);
return $popupReminders;