node grants fails [#859380]

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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

davidtrainer CreditAttribution: davidtrainer commented 20 July 2010 at 13:47

Subscribing

Comment #2

drupalninja99 CreditAttribution: drupalninja99 commented 20 July 2010 at 13:51

File	Size
customreports-nodegrants-patch.txt	5.58 KB

I am adding a patch that replaces the node grants stuff with a simple nodeapi check.

if ($op == 'view') {
      /**
       * Enforce permissions
       */
      if (!user_access('view custom reports')) {
        drupal_access_denied();
        exit;
      }

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.