Statically cache node access grants

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

This could use an issue summary update using the default template for steps to reproduce, proposed solution, and remaining tasks. To help the committers later.

Re-roll

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

Thanks for the reroll @Liam Morland! Code LGTM! Could you perhaps provide it as MR so the tests can run and we can hopefully finish this soon?

Still has a "Needs issue summary update" tag.

It is a MR; see #40.

Sorry @Liam Morland shame on me, I didn't see #40 just #38 -.- Thank you!

Have not tested but see it was previously tagged for issue summary which still appears to be needed. Recommend using standard issue template even if the section doesn't apply, leaving N/A helps reviewers more then it missing.

This needs a reroll

Also drupal_static is being deprecated (see #2260187: [PP-1] Deprecate drupal_static() and drupal_static_reset() → ) so we should not add new usages of it. This would require moving node_access_grants into a service and deprecating the function.