API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Extension...

> ...: Arguments to pass to the hook implementation.

That's a relic of D7.

This now takes a single $args parameter, which is an array of all the parameters.

CommentFileSizeAuthor
#3 parameters_to-2738575-3.patch609 byteslluvigne
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joachim created an issue. See original summary.

cilefen’s picture

cilefen CreditAttribution: cilefen as a volunteer commented
Title: parameter to ModuleHandler::invoke() are incorrect » Parameters to ModuleHandler::invoke() are incorrect

I think this is Major priority for documentation based on the importance of this API.

lluvigne’s picture

lluvigne
Primary language Spanish
Location Seville, Spain
CreditAttribution: lluvigne at La Drupalera by Emergya commented
Status: Active » Needs review
FileSize
parameters_to-2738575-3.patch609 bytes

Hi!
this is my first time on documentation so maybe im doing wrong.
This is what i understood. I changed the doc from where is inherited.

cilefen’s picture

cilefen CreditAttribution: cilefen as a volunteer commented

That seems fine to me.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quick fix

Looks good -- good job on your first documentation patch!

  • catch committed 4ad7ddb on 8.2.x 
    Issue #2738575 by lluvigne: Parameters to ModuleHandler::invoke() are...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x, thanks!

  • catch committed 7c4b577 on 8.1.x 
    Issue #2738575 by lluvigne: Parameters to ModuleHandler::invoke() are...

Status: Fixed » Closed (fixed)

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