See parent issue #3200985: [meta] Fix undesirable access checking on entity query usages for context and test coverage policy.
Taxonomy's TermStorage and VocabularyStorage contain helper methods to get terms by their hierachical position. These methods use entity queries that check access. Whether or not they should do this is debatable, but has tricky backwards compatability questions and is not in scope for this issue: see #2938848: TermStorage::loadParents()/getChildren()/getTree() need an option to specify accessCheck(FALSE).
This issue simply makes explicit the currently implicit accessCheck, in order to unblock the parent issue. Priority is major as this blocks #2785449: It's too easy to write entity queries with access checks that must not have them.
Issue fork drupal-3207477
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:
Comments
Comment #3
andypostI see no other places to fix in the module
Comment #5
catchCommitted 90f67d0 and pushed to 9.2.x. Thanks!