Via doc for hook_nodeapi, only the first two parameters are required. Third and fourth must be defined as options or they cause errors when invoking hook_nodeapi with just the first two.

CommentFileSizeAuthor
#3 diff-nodeapi-1979228.patch622 bytessheena.peng
#2 diff-nodeapi-1979228.patch622 bytessheena.peng

Comments

sheena.peng’s picture

sheena.peng commented
Status: Needs review » Active
sheena.peng’s picture

sheena.peng commented
StatusFileSize
new diff-nodeapi-1979228.patch622 bytes
sheena.peng’s picture

sheena.peng commented
Status: Active » Needs review
StatusFileSize
new diff-nodeapi-1979228.patch622 bytes
sheena.peng’s picture

sheena.peng commented
Status: Needs review » Active
alan d.’s picture

alan d. commented
Status: Active » Fixed

Yes you are correct.

hook_nodeapi(&$node, $op, $a3 = NULL, $a4 = NULL)

Ref: http://api.drupal.org/api/drupal/developer!hooks!core.php/function/hook_...

Committed: 0ffadad on 6.x-2.x. Added a space in the "$page =NULL" to match the coding standards "$a = $b".

Nice, a member for just one day and you have had a commit credit! :)

Status: Fixed » Closed (fixed)

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