Allow all files to be downloaded [#1817552]

Allow all file types to display the download original link, not just images.

Comment	File	Size	Author
#1	media_gallery-all_files_download-1817552-1.patch	1.84 KB	fluffy

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

fluffy CreditAttribution: fluffy commented 19 October 2012 at 11:28

Status:

Active

» Needs review

File	Size
media_gallery-all_files_download-1817552-1.patch	1.84 KB

Patch, removes the image type check and changes the "Download original image" to "Download original file"

Comment #2

Moloc CreditAttribution: Moloc commented 19 October 2012 at 12:41

Status:

Needs review

» Needs work

Local files:
This may be a good idea for local files. With your patch - there will be two links to the files (tested with txt, mp3 and mp4 files) on the detail page. One link to the original path and one link with the download header. This may be a bit confusing. [edit] One of the links may be a display-configuration. Needs more tests. [/edit]

Remote files:
I don't think, that we should support downloading remote files. Just tested a youtube video - after waiting a long time, i got a Timeout.