When file entities are created with the mailhandler module their mime types are stored in uppercase. The mime type check in the file_entity_file_formatter_file_image_view hook implementation only checks for lowercase instances. The small patch included fixes that check.

CommentFileSizeAuthor
#2 uppercase_mime-1395426.patch846 bytesarvinsingla
#1 uppercase_mime-1395426.patch846 bytesarvinsingla

Comments

arvinsingla’s picture

arvinsingla commented
StatusFileSize
new uppercase_mime-1395426.patch846 bytes

The patch file which I forgot to attach

arvinsingla’s picture

arvinsingla commented
StatusFileSize
new uppercase_mime-1395426.patch846 bytes

Sorry, made the previous patch from the file_entity folder. This new patch is done from the media module root folder.

Status: Needs review » Needs work

The last submitted patch, uppercase_mime-1395426.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Is there an issue filed against mailhandler to store mime types as lower case? If it's using the mappings provided by core it should not be possible to have an upper case mime type.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Closed (works as designed)
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Note this was fixed in mailhandler with #1406986: Mime types should be lower case