Media currently implements hook_file_download_access() where it checks for a media entity type and does a user access check.

As there is no longer a media entity type, this hook implementation is no longer necessary.

CommentFileSizeAuthor
#1 remove-file-download-implementation-2120797-1.patch602 bytesdevin carlson

Comments

devin carlson’s picture

devin carlson commented
Status: Active » Needs review
StatusFileSize
new remove-file-download-implementation-2120797-1.patch602 bytes

A patch to remove the hook implementation.

Status: Needs review » Needs work

The last submitted patch, remove-file-download-implementation-2120797-1.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Project: File Entity (fieldable files) » D7 Media
Status: Needs work » Needs review

Fixing project.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

#1: remove-file-download-implementation-2120797-1.patch queued for re-testing.

devin carlson’s picture

devin carlson commented
Status: Needs review » Fixed

@Dave Reid thanks for the help!

Committed #1 to Media 7.x-2.x.

Status: Fixed » Closed (fixed)

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