Problem/Motivation

when running drupal 10, there is the following error where the accesscheck is not checked.

In Query.php line 141:

[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().

CommentFileSizeAuthor
#2 3392161-accessCheck-is-missing.patch502 bytesszeberli

Issue fork warmer_view_mode-3392161

Command icon 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:

About issue forks

Comments

szeberli created an issue. See original summary.

szeberli’s picture

szeberli commented
StatusFileSize
new 3392161-accessCheck-is-missing.patch502 bytes

the accesscheck is installed and is asked to check this

devkinetic’s picture

devkinetic commented

Change record: Access checking must be explicitly specified on content entity queries

sarwan_verma made their first commit to this issue’s fork.

sarwan_verma opened merge request !1

sarwan_verma’s picture

sarwan_verma commented
Status: Needs work » Needs review

Hi devkinetic, I have fixed this issue "Drupal\Core\Entity\Query\QueryInterface::accessCheck() is missing and Replace uses of watchdog_exception() with \Drupal::logger()".created MR.
please review & retest it.
Thanks

devkinetic’s picture

devkinetic commented
Status: Needs review » Needs work

I'd prefer to not bundle these changes together. The logger change should be a separate issue, and it should also use dependency injection.

devkinetic’s picture

devkinetic commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.