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

Comments

joachim created an issue. See original summary.

cilefen’s picture

cilefen 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
commented
Status: Active » Needs review
StatusFileSize
new 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 commented

That seems fine to me.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
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
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.