The subject says it all but just to keep this into context - if the node/1/attachment link points to a text or an image file then instead of the 'normal' behavior of browsers rendering the view for them inside the browser these txt/image files get downloaded.
We found it a bit annoying so worked on a fix to maintain this original behavior while still working with this module.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 1856422-2-maintain-content-type.patch | 1.29 KB | lirantal |
| #1 | 1856422-1-maintain-content-type.patch | 1.17 KB | lirantal |
Comments
Comment #1
lirantal commentedSuggested fix for this issue.
Comment #2
lirantal commentedActually the above will cause the file to be downloaded as 'attachment' so it's not a proper fix.
This new patch will detect if the file is of mime type for plain text/pdf and will open in the browser, otherwise it will force a download (with the correct file name).
Comment #3
ruplHey there, thanks for the report accompanied by a patch! Unfortunately I cannot detect any change in the module's behavior with the patch applied. The header info doesn't change at all when I check in my browser devtools, nor does a PDF load automatically in the browser window.
Could you provide me with some steps to reproduce the outcome you expect?
Comment #4
lirantal commentedYou're welcome :)
It was my mistake to mention the PDF example as that was not taken care of but rather what should open in the browser is a text file or an image. So try uploading a readme.txt or an image.jpg and then fetch it with the attachment links URL and they should open inside the browser instead of being downloaded.
Comment #5
avpadernoI am closing this issue, as Drupal 6 is now not supported.