jMailer: failure when using SingleTo feature and mail() function

[Julien]

I noticed a bug when using SingleTo? feature with mail() function transport in current trunk version (so with updated phpMailer 5.1)

the problem is in class.phpmailer.php at line 648

$params = sprintf("-oi -f %s", $this->Sender);

the error occurs at line 668

$rt = @mail($val, $this->EncodeHeader($this->SecureHeader($this->Subject)), $body, $header, $params);

because $this->Sender may be empty (we're in the "else" block at line 665 where sender is empty or safe_mode does not allow extra params)

In this "else" block, if SingleTo? is false, mail() is called without $params, and works fine.

I suggest removing 5th argument on line 668

patch coming...

I think it must be fixed for jelix 1.2

[Julien]

see  http://bitbucket.org/julieni/jelix-trunk-patches/src/16a61933c3be/ticket-1202-jMailer-SingleTo-bug.patch

see also comment here :  http://sourceforge.net/tracker/index.php?func=detail&aid=2897037&group_id=26031&atid=385707 : bug confirmed by others and same solution to fix...

[laurentj]

pushed :  http://bitbucket.org/jelix/jelix-trunk/changeset/6a70404299bf