Missing access check which is now required on queries

Created on 11 June 2024, 5 months ago

Problem/Motivation

The report doesn't work in D10.2 as entity queries need to explicitly set if an access check is not required.

Drupal\Core\Entity\Query\QueryException: Entity queries must explicitly set whether the query should be access checked or not. See Drupal\Core\Entity\Query\QueryInterface::accessCheck(). in Drupal\Core\Entity\Query\Sql\Query->prepare() (line 141 of core/lib/Drupal/Core/Entity/Query/Sql/Query.php).

Steps to reproduce

- Go to `/admin/reports/entity-count`.
- See error.

Proposed resolution

- State that access check is not required in entity queries.

Remaining tasks

- add accesscheck method

🐛 Bug report

Status

Active

Version

1.2

Component

Code

Created by

🇬🇧United Kingdom very_random_man

Live updates comments and jobs are added and updated live.

Merge Requests

!2Missing access check which is now required on queries
Open
🇬🇧United Kingdom very_random_man
updated 5 months ago

Comments & Activities

Issue created by @very_random_man
Merge request !2Explicitly disable access checks as that is required now. → (Open) created by very_random_man
Status changed to Needs review 5 months ago3:38pm 11 June 2024
Comment 5 months ago →
🇬🇧United Kingdom very_random_man
Ok. I've added the code to explicitly state that access checks aren't required for the entity count report queries.
Comment 1 day ago →
🇮🇳India shashank5563 New Delhi
access check has been added in latest release.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024