TypeError: scheduler_content_moderation_integration_module_implements_alter(): Argument #2 ($hook) must be of type string, int given

Created on 14 March 2025, 3 months ago

Problem/Motivation

In very specific situations (see #3508292), the $hook argument of hook_module_implements_alter() implementations can contain an integer instead of a string.

Steps to reproduce

Install both the pathauto and pathauto_update modules.

Proposed resolution

Change the type hint to allow integers.

🐛 Bug report

Status

Active

Version

3.0

Component

Code

Created by

🇧🇪Belgium dieterholvoet Brussels

Live updates comments and jobs are added and updated live.

Merge Requests

!83TypeError: scheduler_content_moderation_integration_module_implements_alter(): Argument #2 ($hook) must be of type string, int given
Open
🇧🇪Belgium dieterholvoet
updated 3 months ago

Comments & Activities

Issue created by @dieterholvoet
Merge request !83Allow integer hook names → (Open) created by dieterholvoet
Comment 3 months ago →
🇧🇪Belgium dieterholvoet Brussels
Pipeline finished with Success
3 months ago
Total: 382s
#448359
Status changed to Needs review 11 days ago1:03pm 23 May 2025
Comment 11 days ago →
🇨🇭Switzerland berdir Switzerland
As you said, this is IMHO a core bug, especially in this scenario because it's a bug in the BC hook discovery. even if you would define a hook starting with a number, it should cast it to a string.
Comment 11 days ago →
🇧🇪Belgium dieterholvoet Brussels

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024