From dbc58e3dd0fd3bfd670c612f843d52aa0b8664d9 Mon Sep 17 00:00:00 2001 From: nJim Date: Mon, 3 Mar 2025 12:49:16 -0500 Subject: [PATCH] feat(shs-6048): simplify preparer access logic --- docroot/modules/humsci/hs_editorial/hs_editorial.module | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/docroot/modules/humsci/hs_editorial/hs_editorial.module b/docroot/modules/humsci/hs_editorial/hs_editorial.module index 2a8ca7448..bf55a4ff5 100644 --- a/docroot/modules/humsci/hs_editorial/hs_editorial.module +++ b/docroot/modules/humsci/hs_editorial/hs_editorial.module @@ -24,8 +24,7 @@ function hs_editorial_node_access(NodeInterface $node, $op, AccountInterface $ac // not differentiate between published and unpublished nodes. if ($op === 'update' && $node->isPublished() - && in_array('preparer', $account->getRoles()) - && !$account->hasPermission('edit any ' . $node->bundle() . ' content')) { + && in_array('preparer', $account->getRoles())) { return AccessResult::forbidden()->addCacheableDependency($node); } // Default response should return neutral, letting other access checks decide.