From 0e32d35effcc16f515fc0547a874f7017e577db7 Mon Sep 17 00:00:00 2001 From: Simey Lameze Date: Fri, 25 Jul 2014 11:44:44 +0800 Subject: [PATCH] MDL-44257 message: check if the attachment exists before remove it Credit goes to Mark Nielsen, thanks. --- message/output/email/message_output_email.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/message/output/email/message_output_email.php b/message/output/email/message_output_email.php index cfe9fd460cf..4b95db11433 100644 --- a/message/output/email/message_output_email.php +++ b/message/output/email/message_output_email.php @@ -88,7 +88,9 @@ class message_output_email extends message_output { $eventdata->fullmessagehtml, $attachment, $attachname); // Remove an attachment file if any. - @unlink($attachment); + if (!empty($attachment) && file_exists($CFG->dataroot.'/'.$attachment)) { + unlink($CFG->dataroot.'/'.$attachment); + } return $result; }