Problem/Motivation

Scanning the module with the Upgrade status module gives the following warning.
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.
on line 1498 in web/modules/contrib/charts/src/Element/BaseSettings.php

Steps to reproduce

Scan the module with the upgrade status module.

Proposed resolution

Add access check to the entity query.

CommentFileSizeAuthor
#2 add-access-check-3357894-2.patch524 byteshmdnawaz

Issue fork charts-3357894

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

hmdnawaz created an issue. See original summary.

hmdnawaz’s picture

hmdnawaz commented
StatusFileSize
new add-access-check-3357894-2.patch524 bytes

hmdnawaz opened merge request !54

hmdnawaz’s picture

hmdnawaz commented
Status: Active » Needs review

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

  • andileco committed 4a661db8 on 5.0.x authored by hmdnawaz 
    Issue #3357894 by hmdnawaz: Resolve access check error for D-10...
andileco’s picture

andileco commented
Status: Needs review » Fixed

Thank you!

Status: Fixed » Closed (fixed)

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