New hook "Node access records save"

Created on 3 August 2017, over 7 years ago

Updated 26 January 2023, about 2 years ago

In some cases we need to react after calculating and saving node grants.
Hooks "node_access_records" and "node_access_records_alter" are non-useful in this cases.

✨ Feature request

Status

Needs work

Version

10.1 ✨

Component

Node system →

Last updated about 3 hours ago

No maintainer

Created by

🇷🇺Russia kalistos

Live updates comments and jobs are added and updated live.

Needs Review Queue Initiative
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.

Needs tests
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Needs subsystem maintainer review
It is used to alert the maintainer(s) of a particular core subsystem that an issue significantly impacts their subsystem, and their signoff is needed (see the governance policy draft for more information). Also, if you use this tag, make sure the issue component is set to the correct subsystem. If an issue significantly impacts more than one subsystem, use needs framework manager review instead.

Needs change record
A change record needs to be drafted before an issue is committed. Note: Change records used to be called change notifications.

Incomplete comments

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment about 2 years ago →
🇺🇸United States smustgrave
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.

Seems like it could still be relevant for D10
But before it can be added it will need tests and a change record.

Before that I tagged for subsystem review for their thoughts if they can see a reason to not include.

Thanks.
Status changed to Postponed: needs info 28 days ago11:47pm 20 March 2025
Comment 28 days ago →
🇦🇺Australia acbramley
Is this a duplicate of ✨ Add an event or hook that's fired when node access rebuild is completed, or update docs to reflect it's not possible Postponed: needs info ?

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024