Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This has been reported in:
http://drupal.org/node/61528#comment-158819
http://drupal.org/node/30525#comment-166465
This all deals with the same issue: IE is picky about WHAT headers it gets for files and the ORDER of those headers.
In this case, 2 windows appear with the document in both.
This has been tested on:
IE/XP with Office 2003
FF/XP with Office 2003
FF/Linux
hosted on Apache2/PHP4.3/Drupal 5.02beta on Centos 4.4
Enjoy!
Comment | File | Size | Author |
---|---|---|---|
#1 | upload_21.patch | 887 bytes | RobRoy |
upload.module.patch_3.txt | 779 bytes | davea |
Comments
Comment #1
RobRoy CreditAttribution: RobRoy commentedBah, my last message got deleted. Anyways, the patch needs some work. It adds back stuff that we removed in one issue above and some other stuff that's probably unneeded. I stripped down the headers to the bare bones and it looks like this makes PDFs behave EXACTLY the same in public/private files (open up in the current window).
I used web-sniffer.net to see what headers are sent for a public PDF and it looked like just Content-Length and Content-Type were the relevant headers and no name was used for Content-Type (which was causing errors with IE6 when the filename had spaces in it). None of the other stuff. So let's work up from this point to see if any other headers are truly needed for private files.
Please test this out on a variety of private files and maybe this is enough.
Comment #2
RobRoy CreditAttribution: RobRoy commentedCorrecting version. Also, Dave there were some code-style/spacing issues in your patch FYI. Thanks for spearheading this issue, let me know how the latest patch works for you.
Comment #3
drummThe last follow-up with a patch mentioned it needs work.
Comment #4
RobRoy CreditAttribution: RobRoy commentedActually, I was saying Dave's patch had some issues. The patch I posted in #1 is up for review.
Comment #5
Dries CreditAttribution: Dries commentedYes, this patch is needed to make things work. Committed.
Comment #6
(not verified) CreditAttribution: commented