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.My setup:
* vocabulary "category" configured to control access
* vocabulary "version" not configured to control access
* content type with term reference fields to both vocabularies
Limiting access to terms of vocabulary "category" have no effect since taxonomy_tools_build_grants() sets "view"-grants to TRUE because terms in vocabulary "version" do not control access.
Proposed solution:
in taxonomy_tools_build_grants() "view"-grants should be set to FALSE if one of the used conditions say so. Currently it is set to TRUE if one condition grants access ("Grant" overrides "Deny").
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | deny_access-1920170-3.patch | 3.61 KB | SteffenR |
| #1 | deny_access-1920170-2.patch | 3.85 KB | stBorchert |
Comments
Comment #1
stBorchertHeres a rought patch for this.
Comment #2
SteffenRHi stborchert - i rerolled the patch against the latest DEV of the module :)
SteffenR
Comment #3
Shane Birley CreditAttribution: Shane Birley commentedTested by am having trouble with rebuilding the access tables. It sticks about a 1/4 of the way through.
Comment #4
Shane Birley CreditAttribution: Shane Birley commentedI also noticed that the setting for the 403 is also seemingly ignored. Just looking into this to see if it is related to access tables.
Update: It must be related to the permissions failing, doesn't make sense otherwise.
Comment #5
Shane Birley CreditAttribution: Shane Birley commentedApplied patch to the dev and the permission rebuild still fails on line 261.
Comment #6
Shane Birley CreditAttribution: Shane Birley commentedOkay, I have beaten this around and set up a clean Drupal install and I still get the same batch issue.