ISPConfig3 devel
parent: 5cc181a9 | patch | commit | show whitespace
Till Brehm
2015-09-18 d8faa4e7e3fb74e554f6c9339f6fe4e2883c3bf1 
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
1 files modified
2 ■■■■■ changed files
server/lib/classes/monitor_tools.inc.php 2 ●●●●● patch | view | raw | blame | history 
 server/lib/classes/monitor_tools.inc.php


@@ -702,7 +702,7 @@


        $mailSubject = '';


        $inHeader = true;


        for($l = 0; $l < count($lines); $l++) {


            if($lines[$l] == '') {


            if(trim($lines[$l]) == '') {


                $inHeader = false;


                continue;


            }