Problem/Motivation

Error: 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.

Locations:

  • web/modules/contrib/mailchimp/modules/mailchimp_campaign/src/Controller/MailchimpCampaignController.php 390
  • web/modules/contrib/mailchimp/modules/mailchimp_lists/mailchimp_lists.module 435
  • web/modules/contrib/mailchimp/modules/mailchimp_signup/src/Form/MailchimpSignupForm.php 495

Steps to reproduce

Scan the module with the upgrade_status module.

Proposed resolution

Resolve the access check errors.

CommentFileSizeAuthor
#4 3362175_mailchimp_resolve-access-check-4.diff2.16 KBcodebymikey

Issue fork mailchimp-3362175

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 opened merge request !51

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

codebymikey’s picture

codebymikey commented
Version: 2.2.0 » 2.x-dev
Status: Active » Needs review
StatusFileSize
new 3362175_mailchimp_resolve-access-check-4.diff2.16 KB
sagesolutions’s picture

sagesolutions commented
Status: Needs review » Reviewed & tested by the community

Added MR 51 and the errors in the upgrade_status page are now gone. Marking as RTBC

saganakat’s picture

saganakat commented

This patch work for me in D10.2. Thanks!

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

  • xenophyle committed f0659303 on 2.x authored by hmdnawaz 
    Issue #3362175 by hmdnawaz, codebymikey, xenophyle, sagesolutions,...

xenophyle credited aprice42.

xenophyle’s picture

xenophyle commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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