I quickly read through the code in CVS. Looks good.

I don't think you need this function. It seems to be in place for anonymous users and cached pages, but enabling your module should clear the cache and rebuild the permissions to those nodes.

Not sure why this is in forum_access module. I don't use it for Domain Access, but I haven't done extensive testing of node access modules and cache.

og_init() doesn't do anything like this either, and it's the mother of all node access modules.

Comments

harry slaughter’s picture

Assigned: Unassigned » harry slaughter

yeah, i looked at it for a while and just couldn't understand what it was for. cached pages don't circumvent the grants process. don't know why you'd need a special case to deny access.

harry slaughter’s picture

Status: Active » Fixed

ok, removed this from latest version.

shouldn't break anything :)

harry slaughter’s picture

Status: Fixed » Closed (fixed)