From: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Date: Thu, 29 Mar 2012 19:12:21 +0000 (-0400)
Subject: put name= back into Content-Type for IE8
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=9bfd10d62b4f4bbb22e0d580af4d666fb17e62c2;p=evergreen%2Fequinox.git

put name= back into Content-Type for IE8

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
---

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/FlatFielder.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/FlatFielder.pm
index 4cb25cc601..0e5fc9868d 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/FlatFielder.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/FlatFielder.pm
@@ -45,7 +45,11 @@ my $_output_handler_dispatch = {
         "code" => sub {
             my ($r, $args) = @_;
             $r->headers_out->set("Content-Disposition" => "attachment; filename=FlatSearch.csv");
-            $r->content_type('text/csv; charset=utf-8');
+            # Anecdotally, IE 8 needs name= here to provoke downloads, where
+            # other browswers respect filename= in Content-Disposition.  Also,
+            # we might want to make the filename choosable by CGI param later?
+            # Or vary it by timestamp?
+            $r->content_type('text/csv; name=FlatSearch.csv; charset=utf-8');
             print_data_as_csv($args, \*STDOUT);
             return Apache2::Const::OK;
         }