Comment seems wrong.

Ref. http://api.drupal.org/api/drupal/core--modules--node--node.api.php/funct...
Ref. http://api.drupal.org/api/drupal/core--modules--node--node.api.php/funct...

Cheers,

Albert.

CommentFileSizeAuthor
#2 1341724-3-right-hook-name-in-comment-hook_node_view.patch454 bytesalberto56

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Active » Postponed (maintainer needs more info)

Please give a more complete explanation of what you think is wrong, with a suggestion for replacement. For bonus points. please provide a patch.

Thanks,
-Randy

alberto56’s picture

alberto56 commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new 1341724-3-right-hook-name-in-comment-hook_node_view.patch454 bytes

Hi, here's the patch.

Nodeapi is implementing hook_node_view(), but states in the comment that it is implementing hook_view().

Cheers,

Albert.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. Thanks.

alberto56’s picture

alberto56 commented

...which brings me to my next issue, #1342790: Provide example implementation for hook_view()

Albert.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Reviewed & tested by the community » Fixed

Thanks -

7.x: http://drupalcode.org/project/examples.git/commitdiff/1d8a8ab
8.x: http://drupalcode.org/project/examples.git/commitdiff/4616cea

Please do go ahead with a patch for hook_view(). I think it will be useful.

alberto56’s picture

alberto56 commented

thanks @rfay

I've tried for a while to understand how hook_view() works, but to no avail. If I do understand it, I'll submit a patch for sure!

Albert.

Status: Fixed » Closed (fixed)

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