In issue #611772: Node access hooks don't quite make sense... → and identified again in 📌 [meta] Remove the last $op params from our hooks Active , it was suggested to remove the $op from the function, because of the functions already established in hook_node_access_records.
Patch is attached.
Postponed: needs info
11.0 🔥
node system
Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Is this still something we want to do? As per #4 it seems like functionality that might be used, I'm struggling to find examples in gitlab though.
The hooks are fired from the node_access_grants function, that function is called in several places with a hardcoded view operation, but NodeAccesControlHandler does pass in the $operation to NodeGrantDatabaseStorage::access which is passed to node_access_grants.