We tracked down an incompatibility between mailhandler and media module and discovered that media module expects lower case mime types. (EG. "image/png" will display correctly, while "IMAGE/PNG" will not) According to media maintainer Dave Ried, lowercase is what is expected in core. (See issue: #1395426: File entity formatter view doesn't support uppercase mime types )

FYI, our code does a normal mapping from attachment to image field type followed by a hook_node_presave() that will convert the image type to a media field.

Patch to follow.

CommentFileSizeAuthor
#3 mailhandler_mime_type-1406986-3.patch1.79 KBcleaver
#1 mailhandler_mime_type-1406986-1.patch800 bytescleaver

Comments

cleaver’s picture

cleaver commented
Status: Active » Needs review
StatusFileSize
new mailhandler_mime_type-1406986-1.patch800 bytes

Patch against 7.0-2.x.

danepowell’s picture

danepowell commented
Status: Needs review » Fixed

Thanks for the catch. I fixed it in a slightly different way- I made all MIME types lowercase from the outset. Please test the dev release that rolls tonight and let me know if everything works as expected.

http://drupalcode.org/project/mailhandler.git/commit/0602038
http://drupalcode.org/project/mailhandler.git/commit/744a95b

cleaver’s picture

cleaver commented
StatusFileSize
new mailhandler_mime_type-1406986-3.patch1.79 KB

I'm patching 2.4 for testing. Mostly just for our reference, I'm adding the patch file.

cleaver’s picture

cleaver commented

Just a status update to confirm that everything works fine after this change.

Status: Fixed » Closed (fixed)

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