Could possibly be exposed as a local task and action. Would be nice to add this once file_access('download') lands.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Dave Reid’s picture

We should also implement a lot of the functionality in http://drupal.org/project/download_file. Should also trigger headers to cause the browser to trigger a download dialog rather than display the file in-browser.

Jackinloadup’s picture

Here is a reroll of the patch in #1 to work with the #1227706-78: Add a file entity access API for initial testing purposed. Full implementation still needed as stated in #2

Devin Carlson’s picture

Status: Postponed » Needs review
FileSize
3.96 KB

An updated patch to accommodate the changes required due to #1227706: Add a file entity access API and to add in the functionality provided by the DownloadFile module (along with an alter hook to allow other modules to modify the file download headers).

Devin Carlson’s picture

Rerolled and added test coverage.

ParisLiakos’s picture

Status: Needs review » Reviewed & tested by the community

awesome:)
i dont think we need something else here..i just checked this codewise and seems good to go
No manual testing but i trust devin did that already:)

bneil’s picture

I manually tested this and it works for me.

gmclelland’s picture

Just to verify, I did apply the patch in #5. I was able to receive a force download at file/49/download. I'm also seeing the new download permissions.

Devin Carlson’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x-1.x.

This is needed by CKEditor Link File to allow editors to easily link to file "downloads".

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.