D10 compatibility: access checks must be explicitly specified on content entity queries

Created on 16 January 2023, almost 2 years ago
Updated 17 January 2023, almost 2 years ago

Problem/Motivation

See #3201242 β†’ - this was deprecated in 9.2, but has not yet been updated in ConfigPagesController, line 139

AFAIK, this only affects querying content entities, so no need to update the queries that query config entities, e.g. querying for the config page types.

Proposed resolution

Each time ->getQuery() is called to query for a content entity, make sure to follow up with a call to ->accessCheck(). For BC, we should call ->accessCheck() (which is equivalent to ->accessCheck(TRUE)).

πŸ› Bug report
Status

Fixed

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States sonfd Portland, ME

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Issue was unassigned.
  • Status changed to Needs review almost 2 years ago
  • Status changed to Fixed almost 2 years ago
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024