Module does not respond to the mime preference setting (HTML or Plain Text) on the mailbox form in admin. This patch should fix it.

P.s. Look mom, my first patch!

CommentFileSizeAuthor
#3 mailhandler_mime.patch707 bytesz.stolar
mailhandler_mime_0.patch502 bytesvanvemden

Comments

tw99’s picture

tw99 commented

Can't provide an official code review, but the logic looks OK and I can certainly report that this fixes the same problem for me... Thanks.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Needs work

Patch is not applying for me. Perhaps that code has changed already.

z.stolar’s picture

z.stolar commented
StatusFileSize
new mailhandler_mime.patch707 bytes

No - the error is still there.
Here's a revised patch against latest CVS version (1.87.2.13), hopefully it will work now.

z.stolar’s picture

z.stolar commented
Version: 5.x-1.2 » 5.x-1.3
Status: Needs work » Needs review
moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

committed. thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.