CiviMRF / cmrf_core

Drupal Core for CMRF
GNU Affero General Public License v3.0
2 stars 6 forks source link

Add access checks to entity queries to prevent errors in D10 #65

Closed arnalyse closed 1 year ago

arnalyse commented 1 year ago

original deprecation 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.

arnalyse commented 1 year ago

This should be relatively straightforward, as the access check prior to D10 were implicit, so there is no semantic change here, just a more explicit version of writing out the entity query. (This change was introduced to remind developer writing entity queries to always consider access checks, to prevent bugs in production.)

jensschuppe commented 1 year ago

For documentation, this is the relevant change record on drupal.org: https://www.drupal.org/node/3201242

jensschuppe commented 1 year ago

Released with 2.1.0-beta2. Thanks a lot, @arnalyse!