Problem/Motivation

#1984528: Follow-up: Allow for more robust access checking introduced Access(Check)Interface::(ALLOW|DENY|KILL), but didn't apply it to EntityAccessCheckTest.

Proposed resolution

Use AccessCheckInterface::ALLOW instead of TRUE.

Remaining tasks

None.

User interface changes

None.

API changes

None.

(Split into its own issue from #2047659-7: Add test coverage for edit module access checkers at the request of dawehner.)

CommentFileSizeAuthor
#1 2070045-1.patch839 byteswim leers

Comments

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new 2070045-1.patch839 bytes
berdir’s picture

berdir
Primary language German
Location Switzerland
commented

#1: 2070045-1.patch queued for re-testing.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Reviewed & tested by the community

Yes, makes sense.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed e21e628 and pushed to 8.x. Thanks!

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