Problem/Motivation
For the last dependency injection in Http data fetcher plugin, the comma is before the closing parenthesis when it should be right after
Instead of
/**
* {@inheritdoc}
*/
public static function create(ContainerInterface $container, array $configuration, $plugin_id, $plugin_definition): self {
return new static(
$configuration,
$plugin_id,
$plugin_definition,
$container->get('http_client'),
$container->get('plugin.manager.migrate_plus.authentication',)
);
}
We should have
/**
* {@inheritdoc}
*/
public static function create(ContainerInterface $container, array $configuration, $plugin_id, $plugin_definition): self {
return new static(
$configuration,
$plugin_id,
$plugin_definition,
$container->get('http_client'),
$container->get('plugin.manager.migrate_plus.authentication'),
);
}
Comments
Comment #2
danrodComment #4
danrodI took the liberty to create a small change for it, hope you don't mind.
Kindly review: https://git.drupalcode.org/project/migrate_plus/-/merge_requests/138
Comment #5
danrodComment #7
heddnThanks for the fix.