Drupal Answers is a question and answer site for Drupal developers and administrators. It only takes a minute to sign up.
Sign up to join this community
Is there a way to force node access to allowed if there is a hook in a different module marking it as forbidden?
I have a custom module where i implement hook_node_access, returning allowed should certain conditions apply, but the result is being override by a contrib module that sets node access to forbidden.
Can i force my result?
The only way i found to fix this was preventing the conflicting hook from running:
function myModule_module_implements_alter(&$implementations, $hook) {
if ($hook == 'node_access') {
unset($implementations['conflicting_module']); // Not any more
}
}
As it contains useful functionality, i'm running it in my own code, should my conditions for allowed not apply:
function myModule_node_access(NodeInterface $node, $op, AccountInterface $account) {
if ($account->isAuthenticated() and myConditionIsTrue()) {
return AccessResult::allowed();
}
// Run removed hook
if (function_exists('conflicting_module_node_access')) {
return conflicting_module_node_access($node, $op, $account);
}
return AccessResult::neutral();
}
This is less than ideal and i would prefer an alternative.
module_implements_alter, but that opens you up to a higher maintenance burden. I've seen the same pattern implemented in 7 quite a few times if that helps to reassure
– Clive♦
Jul 27 '17 at 17:48
hook_node_access you can try $grants = \Drupal::moduleHandler()->invokeAll('node_access', [$node, $op, $account]);
– florian_drupal
Jul 28 '17 at 5:35
I believe it's best to not remove the conflict implementation of the same hook (Unless you REALLY need that and you know what you are doing). Instead you can follow the Drupal documentation of hook_module_implements_alter and move your hook to the end of the implementations list to override the conflict and not ignore it at all.
function YOURMODULENAME_module_implements_alter(&$implementations, $hook) {
if ($hook == 'node_access') {
// Move my module implementation of node_access to the end of the list.
// \Drupal::moduleHandler()->getImplementations()
// iterates through $implementations with a foreach loop which PHP iterates
// in the order that the items were added, so to move an item to the end of
// the array, we remove it and then add it.
$group = $implementations['YOURMODULENAME'];
unset($implementations['YOURMODULENAME']);
$implementations['YOURMODULENAME'] = $group;
}
}
