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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Wim Leers’s picture

Wim Leers
Location Ghent 🇧🇪🇪🇺
CreditAttribution: Wim Leers commented
Status: Active » Needs review
FileSize
2070045-1.patch839 bytes
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented

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

Berdir’s picture

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

Yes, makes sense.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott 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.