Closed (won't fix)
Project:
Talk
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 Nov 2009 at 10:49 UTC
Updated:
3 Mar 2010 at 12:04 UTC
Jump to comment: Most recent file
Comments
Comment #1
Mark Theunissen commentedComment #2
cwgordon7 commentedWould that be at the expense of PHP 4 compatibility? (If so, we need to add a minimum PHP version statement to the info block).
Comment #3
berdirThe change would in fact break PHP 4 compatibility.
However, using & in hook_nodeapi() is just fine and correct. *If* you recieve a warning, something does call the hook in a wrong way which basically means the the feature never worked.
So you need to find out who is calling that hook wrongly, see #360605-200: PHP 5.3 Compatibility (and my comment above) for more information.
Comment #4
avpadernoThe hook is defined as
hook_nodeapi(&$node, $op, $a3 = NULL, $a4 = NULL)in the API documentation.If something must be changed to make Drupal compatible with PHP 5.3, that must be done at Drupal core level (which it seems it has already done, as I don't get any errors running Drupal on PHP 5.3).
Comment #5
oriol_e9gtagged
Comment #6
berdirI think this can be closed, the error is somewhere else.