diff options
author | Igor Scheller <igor.scheller@igorshp.de> | 2019-04-29 23:17:58 +0200 |
---|---|---|
committer | msquare <msquare@notrademark.de> | 2019-04-30 18:09:08 +0200 |
commit | ca0b2d8b5f650aad37b2d53577379efb4b468044 (patch) | |
tree | b57ae5b4a9c3710223a94be691d682ca5db91a6b | |
parent | 9829e6da390c8a62f322e085cbf5b48a586446a0 (diff) |
Improved email error logging
-rw-r--r-- | includes/helper/email_helper.php | 16 | ||||
-rw-r--r-- | includes/sys_log.php | 9 |
2 files changed, 19 insertions, 6 deletions
diff --git a/includes/helper/email_helper.php b/includes/helper/email_helper.php index 802debaa..bad0d539 100644 --- a/includes/helper/email_helper.php +++ b/includes/helper/email_helper.php @@ -2,6 +2,7 @@ use Engelsystem\Mail\EngelsystemMailer; use Engelsystem\Models\User\User; +use Psr\Log\LogLevel; /** * @param User $recipientUser @@ -13,17 +14,17 @@ use Engelsystem\Models\User\User; function engelsystem_email_to_user($recipientUser, $title, $message, $notIfItsMe = false) { if ($notIfItsMe && auth()->user()->id == $recipientUser->id) { - #return true; + return true; } /** @var \Engelsystem\Helpers\Translator $translator */ $translator = app()->get('translator'); $locale = $translator->getLocale(); - + try { /** @var EngelsystemMailer $mailer */ $mailer = app('mailer'); - + $translator->setLocale($recipientUser->settings->language); $status = $mailer->sendView( $recipientUser->contact->email ? $recipientUser->contact->email : $recipientUser->email, @@ -31,8 +32,15 @@ function engelsystem_email_to_user($recipientUser, $title, $message, $notIfItsMe 'emails/mail', ['username' => $recipientUser->name, 'message' => $message] ); - } catch(Exception $e) { + } catch (Exception $e) { $status = 0; + engelsystem_log(sprintf( + 'An exception occurred while sending a mail to %s in %s:%u: %s', + $recipientUser->name, + $e->getFile(), + $e->getLine(), + $e->getMessage() + ), LogLevel::CRITICAL); } $translator->setLocale($locale); diff --git a/includes/sys_log.php b/includes/sys_log.php index 595e8b55..9ac17dc0 100644 --- a/includes/sys_log.php +++ b/includes/sys_log.php @@ -1,14 +1,19 @@ <?php +use Psr\Log\LoggerInterface; +use Psr\Log\LogLevel; + /** * Write a log entry. * This should be used to log user's activity. * * @param string $message + * @param string $level */ -function engelsystem_log($message) +function engelsystem_log($message, $level = LogLevel::INFO) { $nick = "Guest"; + /** @var LoggerInterface $logger */ $logger = app('logger'); $user = auth()->user(); @@ -16,5 +21,5 @@ function engelsystem_log($message) $nick = User_Nick_render($user); } - $logger->info('{nick}: {message}', ['nick' => $nick, 'message' => $message]); + $logger->log($level, '{nick}: {message}', ['nick' => $nick, 'message' => $message]); } |