During the devlopment process node_access_where_sql lost a parameter (http://cvs.drupal.org/viewcvs/contributions/sandbox/jonbob/node_permissi...) that is neccessary to get nodes that may be accessed by an other than the current user. This is neccessary for modules such as notify or subscriptions to work properly.

CommentFileSizeAuthor
#3 node_15.patch1.62 KBkilles@www.drop.org
node_14.patch1.13 KBkilles@www.drop.org
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JonBob’s picture

JonBob CreditAttribution: JonBob commented

At first glance it looks like that was a mistake, and this is the right way to fix it.

Dries’s picture

Dries CreditAttribution: Dries commented

This needs more work/testing, I think. For example, the function node_access() also passes node_access_grants() a variable called $uid that is not declared.

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented
FileSize
node_15.patch1.62 KB

You are right. I added a third parameter $uid to node_access.

Dries’s picture

Dries CreditAttribution: Dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) CreditAttribution: commented