"core/modules/node/src/Plugin/migrate/source/d6/Node.php" "__construct" method uses variable of type "ModuleHandler" instead of "ModuleHandlerInterface".
This is the __construct definition:
public function __construct(array $configuration, $plugin_id, $plugin_definition, MigrationInterface $migration, StateInterface $state, EntityTypeManagerInterface $entity_type_manager, ModuleHandler $module_handler) {
parent::__construct($configuration, $plugin_id, $plugin_definition, $migration, $state, $entity_type_manager);
$this->moduleHandler = $module_handler;
}
This throws an error when e.g. decorating the module handler.
Use "ModuleHandlerInterface" instead.
Active
11.0 🔥
migration system
Thanks @grevil, code-wise seems quite obvious that the Interface should be used! Guess it's simply old code?
Committed/pushed to 11.x and cherry-picked to 10.5.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.