\Drupal\Core\Extension\ModuleHandlerInterface::invoke() and \Drupal\Core\Extension\ModuleHandlerInterface::invokeAll() document that their second arguments must be arrays, but there is no type hinting. Lack of this caused #2170989: Migrate provides invalid arguments when invoking hooks to remain unnoticed, for instance.

CommentFileSizeAuthor
#6 drupal_2170991_5.patch3.03 KBxano
#1 drupal_2170991_1.patch2.05 KBxano

Comments

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Assigned: xano » Unassigned
Status: Active » Needs review
StatusFileSize
new drupal_2170991_1.patch2.05 KB
xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

For the record: this depends on #2170989: Migrate provides invalid arguments when invoking hooks being fixed first.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Related issues: +#2170989: Migrate provides invalid arguments when invoking hooks

Status: Needs review » Needs work

The last submitted patch, 1: drupal_2170991_1.patch, failed testing.

The last submitted patch, 1: drupal_2170991_1.patch, failed testing.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Needs work » Needs review
StatusFileSize
new drupal_2170991_5.patch3.03 KB

Fixed some of the failures.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Title: Type hint array arguments in ModuleHandlerInterface » Migrate provides invalid arguments when invoking hooks/type hint array arguments in ModuleHandlerInterface
chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community

Good catch.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Since this is just changing type-hinting and fixing a typo, I don't think we need tests for this.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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