Access checking must be explicitly specified on content entity queries

Created on 21 November 2023, about 1 year ago

Updated 7 December 2023, about 1 year ago

Problem/Motivation

Executing an entity query without an access check specified is removed in D10.

Steps to reproduce

Attempt to create a reader_chapter node, results in error:

POST request to: http://web/node/add/reader_chapter?reader_publication_id=212The website encountered an unexpected error. Please try again later.

Drupal\Core\Entity\EntityStorageException: 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\Sql\SqlContentEntityStorage->save() (line 817 of core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php).

Proposed resolution

Add a call to ->accessCheck() on entity queries.

🐛 Bug report

Status

Fixed

Version

2.0

Component

Code

Created by

joshhytr

Live updates comments and jobs are added and updated live.

Drupal 10 compatibility

Merge Requests

!2Access checking must be explicitly specified on content entity queries
Merged
joshhytr
updated about 1 year ago

Comments & Activities

Issue created by @joshhytr
Merge request !2Add calls to accessCheck() on entity queries. → (Merged) created by joshhytr
Status changed to Needs review about 1 year ago1:52pm 21 November 2023
Comment about 1 year ago →
joshhytr
First commit to issue fork.
Comment about 1 year ago →
System Message

scott_euser → committed d2608197 on 2.x authored by JoshHytr →
Issue #3403148: Access checking must be explicitly specified on content...
Status changed to Fixed about 1 year ago4:01pm 7 December 2023
Comment about 1 year ago →
🇬🇧United Kingdom scott_euser
Looks great thanks! I am going to merge this into dev and test more thoroughly with your merge requests combined
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024