Deprecate node_access_view_all_nodes(). Move its functionality in NodeAccessControlHandlerInterface

Test will fail until 🐛 ModuleHandler::resetImplementations should reset $this->invokeMap Active is fixed.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".

This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Can the CR also be updated that drupal_static_reset() with the node_grant key is deprecated.

Done

LGTM

One comment on the MR.

Converted to use a memory cache

Think we missed the 11.2 boat can those be updated to 11.3, reviewed again and closed the rest of the threads as answered.

If you are another contributor eager to jump in, please allow the previous poster(s) at least 48 hours to respond to feedback first, so they have the opportunity to finish what they started!

Thanks! All feedback appears to be addressed