When you hide the user-reference field on the display tab, the module does not fire any node_access related function -- it works only when the field is visible. The original code looks for 'display' setting, which is probably wrong. I changed it to look for 'widget', see the attached patch (manually tested).

CommentFileSizeAuthor
na.patch821 bytesmariusz.slonina
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

seattlehimay’s picture

seattlehimay CreditAttribution: seattlehimay commented

Thank you! Thank you!!

I had been beating my head against the wall, trying to figure out why things used to work, and now don't.

Installed patch, cleared cache, rebuilt permissions--now all is working for me again!!

danielb’s picture

danielb CreditAttribution: danielb commented

Ah good call, thank you for finding this, this code was a guess based on debugged variables, cheers. I'll get to it when I can.

danielb’s picture

danielb CreditAttribution: danielb commented

Actually all that code is dodgy, I think there is a better version of these loops in 'Node access node reference'

danielb’s picture

danielb CreditAttribution: danielb commented
Status: Needs review » Fixed

actually guys this has been inadvertently fixed in the dev version already!

danielb’s picture

danielb CreditAttribution: danielb commented

I've made a new release.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.