Entity queries must explicitly set whether the query should be access checked or not

Created on 10 October 2024, 6 months ago

Followup to 🐛 Issue #3387026: Upgrade status fixes Fixed .

Problem/Motivation

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 142 in /var/www/app/docroot/core/lib/Drupal/Core/Entity/Query/Sql/Query.php).
#0 /var/www/app/docroot/core/lib/Drupal/Core/Entity/Query/Sql/Query.php(81): Drupal\Core\Entity\Query\Sql\Query->prepare()
#1 /var/www/app/docroot/modules/contrib/bridtv/src/BridPlaylistSync.php(164): Drupal\Core\Entity\Query\Sql\Query->execute()

Steps to reproduce

Run Cron job to sync playlists

Proposed resolution

add $query->accessCheck(TRUE); to \Drupal\bridtv\BridEntityResolver::getEntityQuery().

🐛 Bug report

Status

Active

Version

2.0

Component

Code

Created by

🇩🇪Germany osopolar 🇩🇪 GER 🌐

Live updates comments and jobs are added and updated live.

Merge Requests

!15Entity queries must explicitly set whether the query should be access checked or not
Merged
🇩🇪Germany osopolar
updated about 2 months ago

Comments & Activities

Issue created by @osopolar
Merge request !15Issue #3479794: Enable access check in BridEntityResolver → (Merged) created by osopolar
Comment 6 months ago →
🇩🇪Germany osopolar 🇩🇪 GER 🌐
Patch for MR!5 to be used with composer.
Status changed to Needs review about 2 months ago2:03pm 17 February 2025
Comment about 2 months ago →
System Message

osopolar → committed 4a8ab5a8 on 2.0.x
Issue #3479794 by osopolar: Enable access check in BridEntityResolver
Comment about 2 months ago →
🇩🇪Germany osopolar 🇩🇪 GER 🌐
On production for a while.
Comment about 1 month ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024