Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.API page: https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Extension...
It would seem that invoking a hook now needs the params for the hook to be a single array, the same as for theme() calls.
The docs for this interface method still use the old format.
The change record here https://drupal.org/node/1894902 also needs to properly mention this change.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | drupal8.documentation.2095219-4.patch.patch | 545 bytes | hrmoller |
Comments
Comment #1
jhodgdonI don't think the change notice needs to mention this actually -- it's implied by the table of changes, and presumably if the function call is different, people will go to api.d.o and read the docs. But yes, the method docs need an update. Seems like a good novice project.
Comment #2
hrmoller CreditAttribution: hrmoller commentedI'm not sure I understand what this issue is abut. Is it just the docs that needs an update or should the paramters of
::invokeAllbe changed?I'm a novice here so be gentle:)
Comment #3
joachim CreditAttribution: joachim commentedThis is a docs issue, so it's just the function docs that need changing.
> I don't think the change notice needs to mention this actually -- it's implied by the table of changes
It's easily missed!
Comment #4
hrmoller CreditAttribution: hrmoller commentedSo would this be sufficient?
Comment #5
hrmoller CreditAttribution: hrmoller commentedComment #7
hrmoller CreditAttribution: hrmoller commented#4: drupal8.documentation.2095219-4.patch.patch queued for re-testing.
Comment #8
dawehnerPerfect!
Comment #9
jhodgdonIndeed, that is what we need. Thanks!
Comment #10
catchCommitted/pushed to 8.x, thanks!