Split from #2835767-61: Media + REST: comprehensive test coverage for Media + MediaType entity types.

@Wim Leers:

MediaAccessControlHandler does not provide a helpful reason for when access is denied for the view operation

CommentFileSizeAuthor
#2 2905748-2-interdiff.txt1.06 KBAnonymous (not verified)
#2 2905748-2.patch1.88 KBAnonymous (not verified)
#2 2905748-2-test-only.patch841 bytesAnonymous (not verified)

Comments

Anonymous’s picture

Anonymous (not verified) commented

vaplas created an issue. See original summary.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new 2905748-2-test-only.patch841 bytes
new 2905748-2.patch1.88 KB
new 2905748-2-interdiff.txt1.06 KB

Re-uploaded a part of #2835767-46: Media + REST: comprehensive test coverage for Media + MediaType entity types (credit to @Wim Leers for this part) + test.

Anonymous’s picture

Anonymous (not verified) commented
Title: Media::setOwnerId() doesn't return the Media entity » MediaAccessControlHandler does not provide a helpful reason for when access is denied for the view operation

The last submitted patch, 2: 2905748-2-test-only.patch, failed testing. View results

seanb’s picture

seanb
Primary language Dutch
Location Netherlands
commented
Status: Needs review » Reviewed & tested by the community

Looks good! Thanks.

  • catch committed 9c240f9 on 8.5.x 
    Issue #2905748 by vaplas: MediaAccessControlHandler does not provide a...

catch credited catch.

catch’s picture

catch
he/him
Primary language English
commented
Version: 8.5.x-dev » 8.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.5.x and cherry-picked to 8.4.x. Thanks!

  • catch committed 13d1bba on 8.4.x 
    Issue #2905748 by vaplas: MediaAccessControlHandler does not provide a...

Status: Fixed » Closed (fixed)

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