Project

General

Profile

Actions

Patch #3782

closed

broken downloads of attached files after trac import due to missing mime type definitions

Added by he buiss over 14 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Importers
Target version:
Start date:
2009-08-26
Due date:
% Done:

0%

Estimated time:

Description

After importing my Trac data to redmine via "migrate_from_trac" I noticed that several files, which have been attached to tickets, cannot be downloaded correctly due to wrong mime type information.

The problem is that several files have no information in column "file_type" of redmine's "attachment" table. This information is being generated by "Redmine::MimeType.of(filename)" which is described in file /lib/redmine/mime_type.rb .

My suggestion is, to add some more mime types to this definition:

    MIME_TYPES = {
      'text/plain' => 'txt,tpl,properties,patch,diff,ini,readme,install,upgrade',
      'text/css' => 'css',
      'text/html' => 'html,htm,xhtml',
      'text/jsp' => 'jsp',
    ...

The ones, I added, were:

      'application/msword' => 'doc',
      'application/vnd.ms-excel' => 'xls',
      'application/vnd.ms-powerpoint' => 'ppt',
      'application/vnd.ms-powerpoint' => 'pps',
      'application/vnd.oasis.opendocument.text' => 'odt',
      'application/vnd.oasis.opendocument.presentation' => 'odp',
      'application/vnd.oasis.opendocument.spreadsheet' => 'ods',
      'application/pdf' => 'pdf',
      'application/rtf' => 'rtf',
      'application/zip' => 'zip',
      'application/x-rar-compressed' => 'rar',
      'application/x-tar' => 'tar',

A nice collection of mime types can be found here: http://de.selfhtml.org/diverses/mimetypen.htm#uebersicht


Files

new_mime_types.patch (2.89 KB) new_mime_types.patch Support more mime-types Babar O'Cap, 2009-09-21 17:43
Actions #1

Updated by Babar O'Cap over 14 years ago

+1

Actions #2

Updated by Babar O'Cap over 14 years ago

Here a patch with new mime types :
doc, xls, ppt, pps, docx, xlsx, ppsx, pptx, 7zip, odt, ods, odp, csv ...

Actions #3

Updated by Andrea Campi over 14 years ago

+1, this is a small issue but a boring one.

Actions #4

Updated by Jean-Philippe Lang over 14 years ago

  • Status changed from New to Resolved
  • Target version set to 0.9.0

See r3258.

Actions #5

Updated by Andrea Campi over 14 years ago

Actually, I was wrong, it seems for me (or rather, for one of users) Attachment.content_type is not null, but it contains 'text/html' instead of 'application/pdf'.
Off to debug this some more...

The commit does look good though :)

Actions #6

Updated by Berk Demirkır over 14 years ago

r3258 causes Internal Error when downloading mime-typeless files. I've just reverted back and it works fine. (filename was a43)

Actions #7

Updated by Berk Demirkır over 14 years ago

I should note that my database was working on 0.8.7 version of Redmine. I've switched to trunk a week ago. All content_types are blank.

In log files, I was getting this error

ArgumentError (:type option required):
  app/controllers/attachments_controller.rb:43:in `download'

Actions #8

Updated by Jean-Philippe Lang over 14 years ago

Berk Demirkır wrote:

r3258 causes Internal Error when downloading mime-typeless files. I've just reverted back and it works fine. (filename was a43)

Fixed in r3281.

Actions #9

Updated by Jean-Philippe Lang over 14 years ago

  • Status changed from Resolved to Closed

Merged in 0.9-stable in r3286.

Actions

Also available in: Atom PDF