The return value section of the doc for hook_node_access_records() in D7 and D6 should be a bullet list for readability, but it isn't.
http://api.drupal.org/api/function/hook_node_access_records/7

CommentFileSizeAuthor
#2 610066.patch4.45 KBjhodgdon

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Also, it references hook_node_grants(), but there are no () on the function name, which makes it not turn into a link.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new 610066.patch4.45 KB

Here's a patch that cleans up the documentation for hook_node_grants(), and also the companion hook_node_access(). I think it makes more sense than the original.

If accepted, should also be ported to Drupal 6.

dries’s picture

dries commented
Status: Needs review » Fixed

Committed to CVS HEAD. Thanks.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 7.x-dev » 6.x-dev
Status: Fixed » Patch (to be ported)

Needs port to Drupal 6.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Patch (to be ported) » Fixed

These are in the contrib repository in Drupal 6, so I fixed directly there.
http://drupal.org/cvs?commit=280224

Status: Fixed » Closed (fixed)
Issue tags: -Novice

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

Issue tags: +Novice

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.