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 is a well-documented bug: http://support.microsoft.com/kb/316431. The fix is also included down at the bottom.
uc_reports.module needs to be updated with this fix (should remove the no-cache header or headers.). This is around Line 258 of uc_reports.module
thanks
Jason
Comment | File | Size | Author |
---|---|---|---|
#3 | 749812.patch | 1.17 KB | longwave |
Comments
Comment #1
TR CreditAttribution: TR commentedSigh, I hate IE. So to summarize the Microsoft issue, IE is incapable of piping data to a helper application like Office, so it must save the data to disk first then tell Office to open that data. But IE will refuse to write it to disk if you've told it not to "cache" the data, since that would be violating the no-cache directive. That means there's no way to download dynamic data over HTTPS if you're using IE - all data has to be cache-able in order for IE to open it.
That's unacceptable for things like Ubercart file downloads, if we need to control how many times a file is downloaded. Caching reports isn't so much of a problem. I think we can fix this throughout Ubercart by removing the cache control headers and then fooling IE by appending a dynamic URL argument like "?time()" which will make each and every download link unique and therefore IE will never find the data in its cache. I'm going to have to test this on file downloads first before I add this to Ubercart - don't want to break those ...
Comment #2
jasonabc CreditAttribution: jasonabc commentedHey TR - yeah not too many fans of IE over here either :-) It's reasonable to expect that M$'s own browser would be able to pipe data over to their spreadsheet software, but clearly our expectations are set too high here... ;-)
Your summary is absolutely correct - which means that currently the "Export to CSV" link at the bottom of the UberCart Sales & Product Reports generates the errors listed in the Knowledgebase article above. I hadn't thought about file downloads though - yikes. More than happy to install/test patches and report back etc.
many thanks
Jason
Comment #3
longwavePatch attached to fix this for CSV report downloads, and that also corrects a minor spelling mistake in the same function. The same patch file can also be applied to the 5.x branch.
Comment #4
TR CreditAttribution: TR commentedThanks. Committed.