We have some custom classes that when we install this module Drupal\hook_event_dispatcher\HookEventDispatcherModuleHandler replaces \Drupal\Core\Extension\ModuleHandler.
This is the first time I have ever installed a package where our custom code using dependency injection needed updating.
This is one snippet of code I had to change across two files.
/**
* The module handler service.
*
* @var \Drupal\Core\Extension\ModuleHandler
*/
protected ModuleHandler $moduleHandler;
is updated to
/**
* The module handler service.
*
* @var \Drupal\hook_event_dispatcher\HookEventDispatcherModuleHandler
*/
protected HookEventDispatcherModuleHandler $moduleHandler;
Active
4.2
Code
mohd sahzad → made their first commit to this issue’s fork.
It's a best practice to inject the interface instead, as implementation can change.
/**
* The module handler service.
*
* @var \Drupal\Core\Extension\ModuleHandlerInterface
*/
protected \Drupal\Core\Extension\ModuleHandlerInterface $moduleHandler;
Injecting interface means you can swap implementation without changing the code.