MDL-51016 messaging: Fix email attachments on Windows
authorMichael Aherne <michael.aherne@strath.ac.uk>
Thu, 6 Aug 2015 11:53:17 +0000 (12:53 +0100)
committerMichael Aherne <michael.aherne@strath.ac.uk>
Thu, 6 Aug 2015 11:53:17 +0000 (12:53 +0100)
lib/moodlelib.php

index 72b62b0..a91cab4 100644 (file)
@@ -5617,11 +5617,11 @@ function email_to_user($user, $from, $subject, $messagetext, $messagehtml = '',
             // Before doing the comparison, make sure that the paths are correct (Windows uses slashes in the other direction).
             $attachpath = str_replace('\\', '/', $attachmentpath);
             // Make sure both variables are normalised before comparing.
-            $temppath = str_replace('\\', '/', $CFG->tempdir);
+            $temppath = str_replace('\\', '/', realpath($CFG->tempdir));
 
             // If the attachment is a full path to a file in the tempdir, use it as is,
             // otherwise assume it is a relative path from the dataroot (for backwards compatibility reasons).
-            if (strpos($attachpath, realpath($temppath)) !== 0) {
+            if (strpos($attachpath, $temppath) !== 0) {
                 $attachmentpath = $CFG->dataroot . '/' . $attachmentpath;
             }