Closed (fixed)
Project:
Munin monitoring Drupal plugin
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Feb 2011 at 21:04 UTC
Updated:
1 Oct 2012 at 12:30 UTC
I enabled the module and it scribbled the following messages onto the top of my website:-
Warning: Call-time pass-by-reference has been deprecated in /home/robogeek/visforvoltage.org/sites/all/modules/munin/munin_api.module on line 88
Warning: Call-time pass-by-reference has been deprecated in /home/robogeek/visforvoltage.org/sites/all/modules/munin/munin_api.module on line 141
On both lines the code reads:
module_invoke_all('muninplugin_alter', &$plugins);
Presumably the modules are expected to add information to $plugins ... So the fix won't be as simple as removing the ampersand.
If it makes any difference, I'm running on Pressflow.
Comments
Comment #1
disparil commentedSame error with Drupal 6.20 (PHP 5.2)
Comment #2
iamEAP commentedThe following threads contain relevant information:
#353494: Remove node_invoke(), comment_invoke(), etc
#329012: Remove call_user_func_array() from ModuleHandler::invoke() + invokeAll()
Comment #3
Alex Savin commentedShould be fixed in 6.x-1.1.
Switched to drupal_alter() function which be used for altering data.