Even though the implementation of hook_nodeapi() in fieldgroup only performs an action for $op == 'view', which uses the last two arguments ($teaser and $page), the function may be called for any other $op, that may or may not use said last arguments.

This causes a PHP error whenever a third party module calls module_invoke_all('nodeapi', $node, $op); without the last two parameters.

The attached patch adds the proper default arguments, as per the Drupal API.

CommentFileSizeAuthor
#1 fieldgroup-nodeapi.patch474 bytesvkareh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vkareh’s picture

FileSize
474 bytes

Patch did not go through... ?

KarenS’s picture

Status: Needs review » Fixed

Yep. Thanks!

Status: Fixed » Closed (fixed)

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