Commit d8faa4e7 authored by Till Brehm's avatar Till Brehm

Fix by dirkd: Since file() doesn't remove "\n" from the lines read the...

Fix by dirkd: Since file() doesn't remove "\n" from the lines read the condition $lines[$l] == '' never evaluated to true. So $mailHeaders would always contain every line from the template while $mailBody contained none of them.

Together with the bugfix of PHP#68776 [1] disallowing multiple "\n"'s & preceding "\n" in mail()'s $additionalHeaders arg this caused mails not being sent because $mailHeaders would include at least one "\n\n" which was meant to separate the mail headers from the body.

[1] https://bugs.php.net/bug.php?id=68776
parent 5cc181a9
......@@ -702,7 +702,7 @@ class monitor_tools {
$mailSubject = '';
$inHeader = true;
for($l = 0; $l < count($lines); $l++) {
if($lines[$l] == '') {
if(trim($lines[$l]) == '') {
$inHeader = false;
continue;
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment