search

apigee / apigee-edge-drupal

The Apigee Edge module enables you to integrate a Drupal 9 or 8 site with Apigee.
https://www.drupal.org/project/apigee_edge
GNU General Public License v2.0
32 stars 44 forks source link

Relying on entity queries to check access by default is deprecated #841

Closed divya-intelli closed 1 year ago

divya-intelli commented 1 year ago

Is your feature request related to a problem? Please describe.

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

Files affected : modules/contrib/apigee_edge/modules/apigee_edge_teams/src/Controller/TeamApp KeysController.php modules/contrib/apigee_edge/modules/apigee_edge_teams/src/Entity/Storage/Tea mInvitationStorage.php modules/contrib/apigee_edge/modules/apigee_edge_teams/src/ParamConverter/Tea mAppNameConverter.php modules/contrib/apigee_edge/src/Controller/DeveloperAppKeysController.php modules/contrib/apigee_edge/src/Entity/Storage/DeveloperAppStorage.php modules/contrib/apigee_edge/src/ParamConverter/DeveloperAppNameConverter.php modules/contrib/apigee_edge/tests/src/Functional/DeveloperAppUITestTrait.php modules/contrib/apigee_edge/tests/src/Functional/DeveloperAppUITestTrait.php modules/contrib/apigee_edge/tests/src/Functional/QueryTest.php

Additional context

https://www.drupal.org/node/3201242

mxr576 commented 1 year ago

Is this a duplicate of #783? :thinking:

divya-intelli commented 1 year ago

@mxr576 , the mentioned files are not covered in #783. Also wanted to keep separate PRs as we have added Vladimir as co-author in #783.