By mikelutz on
Change record status: 
Published (View all published change records)
Project: 
Drupal core
Introduced in branch: 
8.7.x
Introduced in version: 
8.7.0
Issue links: 
https://www.drupal.org/node/2999906
Description: 

In https://www.drupal.org/node/3001535 we moved the logic used in the Get process plugin directly into the Row class. This made creating an instance of the Get plugin no longer necessary in the MigrationLookup plugin, and as such, the plugin.manager.migrate.process service is no longer injected.

Before:

public function __construct(array $configuration, $plugin_id, $plugin_definition, MigrationInterface $migration, MigrationPluginManagerInterface $migration_plugin_manager, MigratePluginManagerInterface $process_plugin_manager)

$migration = new MigrationLookup($configuration, '', [], $migration_plugin->reveal(), $migration_plugin_manager->reveal(), $proces_plugin_manager->reveal());

Now:

public function __construct(array $configuration, $plugin_id, $plugin_definition, MigrationInterface $migration, MigrationPluginManagerInterface $migration_plugin_manager)

$migration = new MigrationLookup($configuration, '', [], $migration_plugin->reveal(), $migration_plugin_manager->reveal());
Impacts: 
Module developers
Updates Done (doc team, etc.)
Online documentation: 
Not done
Theming guide: 
Not done
Module developer documentation: 
Not done
Examples project: 
Not done
Coder Review: 
Not done
Coder Upgrade: 
Not done
Other: 
Other updates done