I build the module search index, and I get the following error messages, after the batch operations are concluded.

warning: Parameter 1 to menu_nodeapi() expected to be a reference, value given in /Library/WebServer/Documents/dr63/includes/module.inc on line 462.
warning: Parameter 1 to path_nodeapi() expected to be a reference, value given in /Library/WebServer/Documents/dr63/includes/module.inc on line 462.
warning: Parameter 1 to search_nodeapi() expected to be a reference, value given in /Library/WebServer/Documents/dr63/includes/module.inc on line 462.
warning: Parameter 1 to trigger_nodeapi() expected to be a reference, value given in /Library/WebServer/Documents/dr63/includes/module.inc on line 462.

I am using PHP 5.3, but that doesn't cause Drupal to report such errors in other circumstances.

Comments

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 6.x-2.0-rc1 » 6.x-2.x-dev
Issue tags: +PHP 5.3

The error is shown from the development snapshot too, even during cron tasks. In this case I case a more explicit error that says

Warning: Parameter 1 to upload_nodeapi() expected to be a reference, value given in module_invoke() (line 462 of /Library/WebServer/Documents/dr61/includes/module.inc).
Warning: Parameter 1 to book_access_nodeapi() expected to be a reference, value given in module_invoke() (line 462 of /Library/WebServer/Documents/dr61/includes/module.inc).
Warning: Parameter 1 to luceneapi_node_nodeapi() expected to be a reference, value given in module_invoke() (line 462 of /Library/WebServer/Documents/dr61/includes/module.inc).

hook_nodeapi() should not be invoked through module_invoke(), but node_invoke_nodeapi().

cpliakas’s picture

cpliakas commented
Issue tags: -PHP 5.3 +6.x-2.0-rc2

Confirmed. Unfortunately node_invoke_nodeapi() is not an option in this case because using it looses the module association with the extra text added to the index. I need to know the module that added the extra text so I can separate out things like taxonomy terms and comments into separate fields as opposed to having them bundled in the body test. I will review node_invoke_nodeapi() to try to implement my functionality in a similar manner without the nasty errors.

Thanks for all the tough love today. It is very much appreciated.
~Chris

cpliakas’s picture

cpliakas commented
Status: Active » Fixed

Fixed in commit #301006.

Status: Fixed » Closed (fixed)
Issue tags: -6.x-2.0-rc2

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

Issue tags: +6.x-2.0-rc2

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