Missing Access Check

Created on 26 June 2024, 3 days ago
Updated 27 June 2024, 2 days ago

Problem/Motivation

Drupal\Core\Entity\Query\QueryException: Entity queries must explicitly set whether the query should be access checked or not.

Steps to reproduce

Install Drupal 10.3
Install VotingAPI 8.3@beta
Create a vote type
Attempt to delete vote type

Proposed resolution

Add accessCheck() function to query chain

🐛 Bug report
Status

Closed: works as designed

Component

Code

Created by

🇺🇸United States bajah1701

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

Merge Requests

Comments & Activities

  • Issue created by @bajah1701
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.5 + Environment: PHP 7.3 & MySQL 5.7
    last update 3 days ago
    9 pass, 2 fail
  • Pipeline finished with Success
    3 days ago
    Total: 221s
    #208580
  • Status changed to Needs review 3 days ago
  • Status changed to Downport 3 days ago
  • 🇮🇳India MukhtarM

    hi @bajah1701, There is already an MR for this and it's merged in the dev version. please use votingapi:3.x-dev@dev.

    Maybe this needs to be ported to the Beta version.

  • Status changed to Closed: works as designed 3 days ago
  • 🇺🇸United States TR Cascadia
  • 🇮🇳India MukhtarM

    @TR can you please mention the status change to closed also? Why is it not there in the latest version as the OP mentioned the issue version as 8.x-3.0-beta4

  • 🇺🇸United States TR Cascadia

    The access check is not missing. Look at the patch - the access check is done literally three lines above the change made by the OP.

    In Drupal, fixed point releases cannot be changed.
    8.x-3.0-beta4 will never change.

    8.x-3.x-dev is where all the fixes, new features, etc. go. And the next release will be taken from the HEAD for 8.x-3.x.

Production build 0.69.0 2024