Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The module workbench_access_view is revealing a small incompatibility with workbench_access: users of roles that have been assign as a viewers only to a particular editorial section WILL SEE this editorial section in the Editorial section select list in node form when creating or updating a node.
To reproduce:
- Enable modules workbench_access and workbench_access_view
- Create a editorial section taxonomy with:
- Parent A
- Child A1
- Child A2
- Parent B
- Parent A
- Use this taxonomy as editorial section in workbench access module
- Create a role "Editor A1"
- Assign to role "Editor A1" editor permission on section "Child A1"
- Assign to role "Editor A1" viewer permission on section "Parent A"
- Log as a user having the role "Editor A1"
- Visit a node creation page: in the editorial section select list: you will see all sections underneath "Parent A": "Child A1" and "Child A2": BUG : we should not see "Child A2"
This bug is minored by the fact that IF a node has already been assigned to section "Child A2", it won't be editable by a user "Editor A1" in this example.
I think that this is just the way the select list is build that is wrong.
Comment | File | Size | Author |
---|---|---|---|
#1 | workbench_access-unassigned-sections-displayed-in-node-form-2091707-1.patch | 1.48 KB | gabriel.achille |
Comments
Comment #1
gabriel.achille CreditAttribution: gabriel.achille commentedAnd the patch proposal.
It is a quick patch... need probably to be reviewed.
Actually it is a patch against workbench_access module. But because workbench_access_view is just a sandbox, i prefer not polluting workbench_access issues list.
Comment #2
agentrickardI'm not a fan of having module-specific logic in here. I'm also not sure we should even fix this programatically, as it seems a bit more like a configuration mistake.
Comment #3
gabriel.achille CreditAttribution: gabriel.achille commentedok, module specific logic here is not great. I will try to find another solution, and check my configuration.