diff --git a/lib/MailQueueHandler.php b/lib/MailQueueHandler.php index 5e64a5b79..3990dde40 100644 --- a/lib/MailQueueHandler.php +++ b/lib/MailQueueHandler.php @@ -132,6 +132,7 @@ public function sendEmails($limit, $sendTime, $forceSending = false, $restrictEm // No users found to notify, mission abort return 0; } + $this->logger->debug('Sending notification emails to users: '. implode(',', $affectedUsers)); $userLanguages = $this->config->getUserValueForUsers('core', 'lang', $affectedUsers); $userTimezones = $this->config->getUserValueForUsers('core', 'timezone', $affectedUsers); @@ -177,6 +178,7 @@ public function sendEmails($limit, $sendTime, $forceSending = false, $restrictEm } $this->activityManager->setRequirePNG(false); + $this->logger->debug('Email notifications sent. Will delete: ' . implode(', ', $deleteItemsForUsers)); // Delete all entries we dealt with $this->deleteSentItems($deleteItemsForUsers, $sendTime); @@ -405,6 +407,12 @@ protected function sendEmailToUser($userName, $email, $lang, $timezone, $maxTime try { $this->mailer->send($message); } catch (\Exception $e) { + $this->logger->logException($e, [ + 'message' => 'Failed sending email to user "{user}"', + 'user' => $user, + 'app' => 'activity', + ]); + $this->activityManager->setCurrentUserId(null); return false; } @@ -498,6 +506,8 @@ protected function deleteSentItems(array $affectedUsers, $maxTime) { $query->delete('activity_mq') ->where($query->expr()->lte('amq_timestamp', $query->createNamedParameter($maxTime, IQueryBuilder::PARAM_INT))) ->andWhere($query->expr()->in('amq_affecteduser', $query->createNamedParameter($affectedUsers, IQueryBuilder::PARAM_STR_ARRAY), IQueryBuilder::PARAM_STR)); - $query->execute(); + $this->logger->debug('Delete SQL: ' . $query->getSQL()); + $result = $query->execute(); + $this->logger->debug('Delete Statement returned: ' . $result); } }