D10 Compatibility: entity queries must check access

Is there an explicit reason why not to use accessCheck(TRUE), which is what is recommended to use by default? does TRUE break something? If not, I'd suggest starting with trying TRUE in most cases to get it committed, and then continue testing and if it breaks something, create a follow-up issue in the future.

We need to load all the entities which is why accessCheck(FALSE) is added. Do we need to load only the entities the user has access to? If yes in that case we need to use the accessCheck(TRUE).

Thanks for the explanation. It makes sense, particularly after looking at more of the context in the diff:

https://git.drupalcode.org/project/address/-/merge_requests/26/diffs

I've updated the issue summary to explain the choice of why to use FALSE in the Access Check condition.

Raising this to Major since it throws a fatal error in Drupal 10.

Thanks, everyone!

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