node_invoke() has too many parameters [#1036080]

node_invoke() is apparently never called with more than 2 parameters: $node and $hook, at least in core.

It should probably be reduced to these 2 parameters: this looks like a remnant from older Drupal versions.

Comment	File	Size	Author
#1	node_1036080.patch	1.05 KB	fgm

Comments

fgm

French

Paris, France

commented 22 January 2011 at 11:07

Status:

Active

» Needs review

Status	File	Size
new	node_1036080.patch	1.05 KB

Sending to test bot to make sure.

jacob.embree commented 15 April 2018 at 21:27

Issue summary:	View changes
Status:	Needs review	» Closed (won't fix)

The additional parameters are available for legacy reasons. One of them is used, for instance, by Ctools (http://www.drupalcontrib.org/api/drupal/contributions%21ctools%21plugins...).

This would be an incompatible API change for Drupal 7.
Drupal 8 removed node_invoke() (https://www.drupal.org/node/2029519).