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.
We've tried this module on a large site and the node grants hoses the entire grants table. I understand why you can't use hook_access(), I don't suppose there's be a way to use hook_nodeapi() to enforce permissions?
Comment | File | Size | Author |
---|---|---|---|
#2 | customreports-nodegrants-patch.txt | 5.58 KB | drupalninja99 |
Comments
Comment #1
davidtrainer CreditAttribution: davidtrainer commentedSubscribing
Comment #2
drupalninja99 CreditAttribution: drupalninja99 commentedI am adding a patch that replaces the node grants stuff with a simple nodeapi check.
This seems to do the trick much more easily without messing with node grants. Understand that on a site with 10's of thousands of nodes, that using node grants is an extremely costly operation and that's if it works.
Comment #3
a_c_m CreditAttribution: a_c_m commentedThis looks good, i wil try to make time at or on DrupalCon to get this committed.
Comment #4
moshe weitzman CreditAttribution: moshe weitzman commentedI'll second that node access system is quite a big unwelcome surprise for sites with lots of nodes. Patch looks good.
Comment #5
a_c_m CreditAttribution: a_c_m commentedCommitted.