BUG (?): vlibMimeMail Reply-To

[interkan]

Took some hunting for me to locate this, but the class has worked great up until now.

If you set the reply-to address for an e-mail by either the replyTo method or by setting $object->xheaders['Reply-To'] = "[email protected]" the reply-to address isn't built in.  I finally found the problem.

On about line 477:

$rgx = ($this->apply_windows_bugfix) ? 'Subject' : 'Subject|To';

The above regular expression also matches the "To" in "Reply-To" as well as "To".  The line should be changed to:

$rgx = ($this->apply_windows_bugfix) ? '^Subject' : '^Subject|^To';

This will search for headers which start with "Subject" or "To" (though it should probably be an exact match with '^Subject$|^To$' as the match criteria to be on the safe side).

Keep up the great work...

[releasedj]

Thank you.

I will apply the fix after new years and release a new version.

Many thanks,

Kelvin.

[Florian]

Hello,

this bug is still not fixed!
The lines 477 and 478 are:

Code: [Select]

$rgx = ($this->apply_windows_bugfix) ? Subject : Subject|To; // dont strip out To header for bugfix
if (!preg_match(/^.$rgx.$/i, $head)) $this->headers .= $head.: .strtr($value, "\r\n", )."\n";
And they have to be changed to:

Code: [Select]

$rgx = ($this->apply_windows_bugfix) ? ^Subject$ : ^Subject$|^To$; // dont strip out To header for bugfix
if (!preg_match(/.$rgx./i, $head)) $this->headers .= $head.: .strtr($value, "\r\n", )."\n";

Thanks,
Florian

[ClausVB]

Thank you!

I will do the bugfix and release the new version in the next few weeks.

Regards,
Claus