Problem/Motivation

See Drupal core Change record - https://www.drupal.org/node/3201242

Webhooks,  2.0.0-alpha1
Scanned on Mon, 05/22/2023 - 13:20

FILE: modules/contrib/webhooks/modules/webhook/src/WebhookListBuilder.php

STATUS         LINE                           MESSAGE
--------------------------------------------------------------------------------
Check manually 68   Relying on entity queries to check access by default is
                    deprecated in drupal:9.2.0 and an error will be thrown from
                    drupal:10.0.0. Call
                    \Drupal\Core\Entity\Query\QueryInterface::accessCheck() with
                    TRUE or FALSE to specify whether access should be checked.
--------------------------------------------------------------------------------

FILE: modules/contrib/webhooks/src/Commands/WebhooksCommands.php

STATUS         LINE                           MESSAGE
--------------------------------------------------------------------------------
Check manually 127  Relying on entity queries to check access by default is
                    deprecated in drupal:9.2.0 and an error will be thrown from
                    drupal:10.0.0. Call
                    \Drupal\Core\Entity\Query\QueryInterface::accessCheck() with
                    TRUE or FALSE to specify whether access should be checked.
--------------------------------------------------------------------------------

Steps to reproduce

Install Upgrade status module.
Execute command drush upgrade_status:analyze webhooks

Proposed resolution

Add ::accessCheck()

CommentFileSizeAuthor
#4 3361865-4-d10-access-check.patch1.09 KBdeimos

Issue fork webhooks-3361865

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

deimos created an issue. See original summary.

deimos’s picture

deimos CreditAttribution: deimos at DevBranch, Drupal Ukraine Community, Evolving Web commented
Issue summary: View changes

deimos opened merge request !3

deimos’s picture

deimos CreditAttribution: deimos at DevBranch, Drupal Ukraine Community, Evolving Web commented
FileSize
3361865-4-d10-access-check.patch1.09 KB

Attached patch file.

deimos’s picture

deimos CreditAttribution: deimos at DevBranch, Drupal Ukraine Community, Evolving Web commented
Assigned: deimos » Unassigned
Status: Needs work » Needs review

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

  • sanduhrs committed 7a9ad4ae on 2.0.x authored by deimos 
    Issue #3361865: D10 compatibility: Relying on entity queries to check...
sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs as a volunteer commented
Status: Needs review » Fixed

Thank you.

Status: Fixed » Closed (fixed)

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