API page: https://api.drupal.org/api/drupal/core%21modules%21node%21node.api.php/f...

The doxygen on hook_node_access_records() refers to node_access_write_grants(), which no longer exists per node_access_write_grants() now named _node_access_write_grants(). Since the purpose of the rename was to discourage the function's use, I assume the correct course of action here is to remove the mention of it rather than to fix it. Patch to follow.

CommentFileSizeAuthor
#2 2664396-2.patch557 bytestraviscarden

Comments

TravisCarden created an issue. See original summary.

traviscarden’s picture

traviscarden commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 2664396-2.patch557 bytes

Note: There was an incomplete @return directive on the line above the change, so I went ahead and fixed that, too, since it's trivial and non-controversial.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks!

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs work

Doesn't apply to 8.1.x.

  • catch committed af30226 on 
    Issue #2664396 by TravisCarden: hook_node_access_records() doxygen...
catch’s picture

catch
he/him
Primary language English
commented
Status: Needs work » Fixed

OK that's because we already fixed this issue in 8.1.x, so committed/pushed to 8.0.x only, thanks!

Status: Fixed » Closed (fixed)

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