Commit 9efdf77d authored by gerv%gerv.net's avatar gerv%gerv.net

Bug 230293 - Send CSV buglists with "Content-Disposition: attachment". Patch by gerv; r,a=justdave.

parent 2eae8e3a
...@@ -866,6 +866,7 @@ if ($dotweak) { ...@@ -866,6 +866,7 @@ if ($dotweak) {
# Generate HTTP headers # Generate HTTP headers
my $contenttype; my $contenttype;
my $disp = "inline";
if ($format->{'extension'} eq "html") { if ($format->{'extension'} eq "html") {
my $cookiepath = Param("cookiepath"); my $cookiepath = Param("cookiepath");
...@@ -894,6 +895,12 @@ else { ...@@ -894,6 +895,12 @@ else {
$contenttype = $format->{'ctype'}; $contenttype = $format->{'ctype'};
} }
if ($format->{'extension'} eq "csv") {
# We set CSV files to be downloaded, as they are designed for importing
# into other programs.
$disp = "attachment";
}
if ($serverpush) { if ($serverpush) {
print $cgi->multipart_start(-type=>$contenttype); print $cgi->multipart_start(-type=>$contenttype);
} else { } else {
...@@ -901,7 +908,7 @@ if ($serverpush) { ...@@ -901,7 +908,7 @@ if ($serverpush) {
# If we are doing server push, then we did this already in the HTTP headers # If we are doing server push, then we did this already in the HTTP headers
# that started the server push, so we don't have to do it again here. # that started the server push, so we don't have to do it again here.
print $cgi->header(-type => $contenttype, print $cgi->header(-type => $contenttype,
-content_disposition => "inline; filename=$filename"); -content_disposition => "$disp; filename=$filename");
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment