Issue fork drupal-3207961

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jonathanshaw created an issue. See original summary.

jonathanshaw’s picture

Status: Active » Needs review
jonathanshaw’s picture

jonathanshaw’s picture

These changes are all trivial because they are to TRUE outside of tests and to FALSE in tests:
- TRUE is the existing default, so specifying it explicitly has no functional impact
- FALSE is the correct choice 99% of the time in tests (see #3204163: EntityQuery accessCheck: tests should not check access unless relevant)

longwave’s picture

Status: Needs review » Reviewed & tested by the community

All the cases have been changed as per #5; production code stays TRUE as before, and only tests change to FALSE which is fine.

  • catch committed 9c226d4 on 9.2.x
    Issue #3207961 by jonathanshaw, longwave: EntityQuery accessCheck:...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.2.x, thanks!

Status: Fixed » Closed (fixed)

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