]> git.ipfire.org Git - thirdparty/bugzilla.git/commitdiff
Fix for bug 111522: Provide ability to specify MIME type of attachment
authorkiko%async.com.br <>
Wed, 29 Oct 2003 09:58:13 +0000 (09:58 +0000)
committerkiko%async.com.br <>
Wed, 29 Oct 2003 09:58:13 +0000 (09:58 +0000)
when downloading. Adds a 'ctype' argument to attachment.cgi which allows
one to override the content-type when viewing an attachment. Original
patch by Alex Vincent <ajvincent@juno.com>; I changed it a tiny bit.
r=kiko, myk. a=justdave.

attachment.cgi

index d020b4f57c2f1d3055652741364f106a9ebb5573..1f855d367a8903b3b3c8142669f74fa13a089091 100755 (executable)
@@ -22,6 +22,7 @@
 #                 Myk Melez <myk@mozilla.org>
 #                 Daniel Raichle <draichle@gmx.net>
 #                 Dave Miller <justdave@syndicomm.com>
+#                 Alexander J. Vincent <ajvincent@juno.com>
 
 ################################################################################
 # Script Initialization
@@ -433,6 +434,15 @@ sub view
     # Retrieve the attachment content and its content type from the database.
     SendSQL("SELECT mimetype, filename, thedata FROM attachments WHERE attach_id = $::FORM{'id'}");
     my ($contenttype, $filename, $thedata) = FetchSQLData();
+   
+    # Bug 111522: allow overriding content-type manually in the posted $::FORM.
+    if ($::FORM{'ctype'})
+    {
+        $::FORM{'contenttypemethod'} = 'manual';
+        $::FORM{'contenttypeentry'} = $::FORM{'ctype'};
+        validateContentType();
+        $contenttype = $::FORM{'ctype'};
+    }
 
     # Return the appropriate HTTP response headers.
     $filename =~ s/^.*[\/\\]//;