Problem/Motivation

To reproduce, try creating a commerce order aggregate sensor, that will work when manually testing but not when running the query as anonymous user because then the query alters from commerce kicks in.

Proposed resolution

Add $query->accessCheck(FALSE); to the entity query.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 3114259-content-entity-aggregate-no-access-check-3-2-interdiff.txt647 bytesmbovan
#3 3114259-content-entity-aggregate-no-access-check-3.patch2.19 KBmbovan
#3 3114259-content-entity-aggregate-no-access-check-3-TEST-ONLY.patch647 bytesmbovan
#2 3114259-content-entity-aggregate-2.patch1.56 KBmbovan

Comments

Berdir created an issue. See original summary.

mbovan’s picture

mbovan commented
Status: Active » Needs review
StatusFileSize
new 3114259-content-entity-aggregate-2.patch1.56 KB

We have to do this for both entity query and entity query verbose.

mbovan’s picture

mbovan commented
StatusFileSize
new 3114259-content-entity-aggregate-no-access-check-3-TEST-ONLY.patch647 bytes
new 3114259-content-entity-aggregate-no-access-check-3.patch2.19 KB
new 3114259-content-entity-aggregate-no-access-check-3-2-interdiff.txt647 bytes

Added tests in Drupal\Tests\monitoring\Functional\MonitoringCommerceTest.

The last submitted patch, 3: 3114259-content-entity-aggregate-no-access-check-3-TEST-ONLY.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

  • Berdir committed 56a7381 on 8.x-1.x authored by mbovan 
    Issue #3114259 by mbovan: Content entity aggregate sensor does not skip...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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