Problem/Motivation

This is a follow-up issue to #3296998: Automated Drupal 10 compatibility fixes (FIXED), as not all compatibility issues were fixed there. The access check for queries is still missing.

Steps to reproduce

Proposed resolution

Add an access check to the queries.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork drowl_paragraphs-3350595

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Grevil created an issue. See original summary.

Grevil opened merge request !53

Grevil’s picture

Grevil CreditAttribution: Grevil as a volunteer and at DROWL.de commented
Status: Active » Needs review

OK, this was only an issue inside an update hook of a submodule, but should still be fixed here! Please review.

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Needs review » Reviewed & tested by the community

Well done, thanks @Grevil!

  • Anybody committed e7cf8bb6 on 4.x authored by Grevil 
    Issue #3350595 by Grevil: Fix missing query access check
Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody at DROWL.de commented
Status: Reviewed & tested by the community » Fixed

Will be part of the next release. No extra release needed.

Status: Fixed » Closed (fixed)

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