diff options
| author | Mark Longair <mhl@pobox.com> | 2013-12-02 16:11:31 +0000 |
|---|---|---|
| committer | Mark Longair <mhl@pobox.com> | 2013-12-03 16:14:03 +0000 |
| commit | 723d6c1d73176bf723b0c4726a150b1842bedf8e (patch) | |
| tree | 6f6542dc99ca3736ba34102aab344655d73c1bd2 /script/rails-deploy-before-down | |
| parent | 0fe4865455786164c95766f514b4625e6061136f (diff) | |
Treat any document/pdf attachment as if it were application/pdf
This is a fix for issue #1232. Richard Taylor pointed
out that some PDF attachments had the non-standard content-type
document/pdf, and that these weren't being treated as PDFs.
(Ganesh Sittampalam discovered that all of these PDFs were
generated by a Lexmark X945e, according to the PDF metadata.)
This commit adds an extra case to normalise_content_type to map
document/pdf to application/pdf.
In fact, since the upgrade of the Mail gem in ccebe3c3d6d4dc5f81
the behaviour when handling the non-standard content-type
document/pdf was much better, but this commit also means that
you get the right icon for the attachment, and can be
cherry-picked onto older versions to fix #1232.
Diffstat (limited to 'script/rails-deploy-before-down')
0 files changed, 0 insertions, 0 deletions