Details
-
Bug
-
Status: Closed
-
Minor
-
Resolution: Fixed
-
3.10
-
MOODLE_310_STABLE
-
MOODLE_310_STABLE
-
MDL-70163-311 -
Ensure you have a test site with e-mail configured, or with a service such as Mailhog or Mailcatcher
- Please MDL-70163.php
in your dirroot directory - From CLI, execute it
$ php MDL-70163.php - Confirm no PHP warning is triggered
- Confirm you receive an e-mail containing a "Hello.txt" attachment containing the text "Test attachment"
Automated tests
- Execute unit tests, confirm they still pass
$ mdk phpunit -r -u lib/tests/moodlelib_test.php
Ensure you have a test site with e-mail configured, or with a service such as Mailhog or Mailcatcher Please MDL-70163.php in your dirroot directory From CLI, execute it $ php MDL-70163.php Confirm no PHP warning is triggered Confirm you receive an e-mail containing a "Hello.txt" attachment containing the text "Test attachment" Automated tests Execute unit tests, confirm they still pass $ mdk phpunit -r -u lib/tests/moodlelib_test.php - Please MDL-70163.php
Description
Related to MDL-66928 and from recent fix to allow email attachments to be included from that directory in MDL-70130
If calling code calls email_to_user and $CFG->localrequestdir hasn't yet been created (/tmp/requestdir on my system), then a PHP warning is generated:
strpos(): Empty needle
|
|
|
/mnt/export/workplace/master/src/lib/moodlelib.php:6338
|
Attachments
Issue Links
- has been marked as being related by
-
MDL-69724 Add a mail filtering and mutation api
-
- Reopened
-
