Posted by Colin Guthrie (coling) on 2011-01-20T06:37:29.000+0000
Patch to implement this new feature.
Posted by Colin Guthrie (coling) on 2011-01-20T07:21:14.000+0000
I noticed a typo in my original code after a refactor... count() was still included after pushing it out to a variable rather than checking the array directly. Very minor change.
Posted by Colin Guthrie (coling) on 2011-03-09T04:21:27.000+0000
Ping? Some kind of feedback would be nice :)
Posted by Michael Kliewe (mkliewe) on 2011-04-04T13:59:42.000+0000
Thank you Colin for this nice patch, I also had the problem to attach inline images, and your code produces mails that are more compatible to mail clients.
Posted by Paul Brand (paulb) on 2011-04-12T08:14:36.000+0000
Thanks a lot, Colin. Your patch solved an issue with inline images in a HTML mail in Mail on OSX.
Posted by Walter Hop (lfms) on 2011-05-05T20:15:57.000+0000
The author of this patch is a god amongst men.
Posted by Colin Guthrie (coling) on 2011-05-05T20:55:41.000+0000
While I appreciate the kudos, I believe gods have the ability to commit to upstream subversion, but sadly I do not. I cannot even smite the maintainer for not acknowledging it thus far... :p
Posted by Tche (tche) on 2011-09-01T11:29:59.000+0000
Very good. Thanks Colin. It saved my day. Multipart/related is definitely required for some mail clients (Thunderbird at least).
I just had one issue when patching in Eclipse.
If you get the error "Call to a member function getParts() on a non-object in ... Abstract.php on line 239". It just means that the patch partially succeeded on Mail.php file.
You have to add this line to the __contruct():
$this->_htmlRelatedAttachments = new Zend_Mime_Message();
Comments
Posted by Colin Guthrie (coling) on 2011-01-20T06:37:29.000+0000
Patch to implement this new feature.
Posted by Colin Guthrie (coling) on 2011-01-20T07:21:14.000+0000
I noticed a typo in my original code after a refactor... count() was still included after pushing it out to a variable rather than checking the array directly. Very minor change.
Posted by Colin Guthrie (coling) on 2011-03-09T04:21:27.000+0000
Ping? Some kind of feedback would be nice :)
Posted by Michael Kliewe (mkliewe) on 2011-04-04T13:59:42.000+0000
Thank you Colin for this nice patch, I also had the problem to attach inline images, and your code produces mails that are more compatible to mail clients.
Posted by Paul Brand (paulb) on 2011-04-12T08:14:36.000+0000
Thanks a lot, Colin. Your patch solved an issue with inline images in a HTML mail in Mail on OSX.
Posted by Walter Hop (lfms) on 2011-05-05T20:15:57.000+0000
The author of this patch is a god amongst men.
Posted by Colin Guthrie (coling) on 2011-05-05T20:55:41.000+0000
While I appreciate the kudos, I believe gods have the ability to commit to upstream subversion, but sadly I do not. I cannot even smite the maintainer for not acknowledging it thus far... :p
Posted by Tche (tche) on 2011-09-01T11:29:59.000+0000
Very good. Thanks Colin. It saved my day. Multipart/related is definitely required for some mail clients (Thunderbird at least).
I just had one issue when patching in Eclipse. If you get the error "Call to a member function getParts() on a non-object in ... Abstract.php on line 239". It just means that the patch partially succeeded on Mail.php file. You have to add this line to the __contruct(): $this->_htmlRelatedAttachments = new Zend_Mime_Message();