Description: ------------ After calling Mail_mime::headers() the first time with some extra headers they cannot be overwritten with a second call. I'd need this to send mails to different recipients with their email address (and maybe their name) in To. Mail_smtp doesn't put the recipients in a To header itself. IMHO $xtra_headers should not be put into which would solve my problem. Reproduce code: --------------- <?php // include ... $mime = new Mail_mime(); $mime->setTXTBody(/* get some text */); $headers = array('To' => 'foo@example.com', 'From' => 'test@example.com', 'Subject' => 'foo and bar'); $body = $mime->get(); $headers = $mime->headers($headers); // Mail::factory() ....->send('text@example.com', $headers, $body) $headers = array('To' => 'bar@example.com', 'From' => 'test@example.com', 'Subject' => 'foo and bar'); $body = $mime->get(); $headers = $mime->headers($headers); // Mail::factory() ....->send('text@example.com', $headers, $body) ?> Expected result: ---------------- // First mail To: foo@example.com From: test@example.com Subject: foo and bar the text . // Second mail To: bar@example.com From: test@example.com Subject: foo and bar the text . Actual result: -------------- // First mail To: foo@example.com From: test@example.com Subject: foo and bar the text . // Second mail To: foo@example.com From: test@example.com Subject: foo and bar the text .

I know this might be just a feature request, but it would be quite easy to implement it. Maybe you could include an option like in: function &headers($xtra_headers = null, $save_xtra_headers = true) { $headers['MIME-Version'] = '1.0'; if (isset($xtra_headers)) { $headers = array_merge($headers, $xtra_headers); } $headers = array_merge($headers, $this->_headers); if($save_xtra_headers) { $this->_headers = $headers; } return $this->_encodeHeaders($headers); } It wouldn't break anything, but would do what I (and maybe others need). The only way to do this now is $old_headers = $mime->_headers; $headers = $mime->headers($my_headers); $mime->_headers = $old_headers; The new function header(...) would allow a simple $headers = $mime->headers($my_headers, false); which is shorter and much cleaner. b4n

After trying to implement a newsletter engine using Mail_mime I came accross this limitation. Is there any practical workaround to make the TO: in each email to be representing the actual instance in the Database instead of the address in the first row? Understanding that keeping backwards compatibility is an issue, it would be fantastic to have some implementation that would allow the use of Mail_mime to send succesive emails (changing the recipient address in the headers) using some class notation instead of "unclean" code.

I came across this bug and it took me hours to drill down and finally discover the culprit: in the headers() method I replaced the first line with the second. basically, swap the array_merge() parameter order so that your custom headers overwrite the internal array. ------------------- //$this->_headers = array_merge($headers, $this- >_headers); $this->_headers = array_merge($this->_headers, $headers); ------------------- IMHO, this is a bug because when a function says that you can set headers and it silently drops them in favor of the first ones it got, the function is implmented poorly. Either that or it is named poorly. I cannot think of any reason why this rewrite would break Backwards Compatibility... who relies on a function that overwrites parameters that you explicitly send it? Please fix this and save some other soul some lunch time. DC

This nasty thing took me too much time. The default behavior is not what one would expect from the function headers(). If you argue this is for backward compatibility i'm wondering why the setHTMLBody() and setTXTBody() functions are working like expected? For what reason would s.o. change the mail-body but keep the headers? Please change this behavior.

The function I wrote here in the comments allows the (other) expected behavior and is backward compatible. Adding it would also document the current behavior and save some WTFs. If the developers of Mail_mime would prefer a patch I'll get HEAD and add that.

Yes, developers, please accept a patch from this generous soul above... Save future generations from the misery with which we have dealt.

I ran in to the same problem and came to the same bugfix. Please accept the patch. The patch leads to the "expected" behavior!

Ugly but working workaround: unset($mime->_headers['To']); $headers = $mime->headers($headers);