Migration plugin throws PHP type error

@chlandin

Please see attached patch. Let me know if this works for you.

Thanks,
George

Thanks for the quick reply @geoanders.

I'm afraid your patch is not compatible with version 2.3 that I got via composer require.
If you look here you can see that all arguments in __construct() are on the same line.

Thanks for the quick reply @geoanders.

I'm afraid your patch is not compatible with version 2.3 that I got via composer require.
If you look here you can see that all arguments in __construct() are on the same line.

Thanks for the quick reply @geoanders.

I'm afraid your patch is not compatible with version 2.3 that I got via composer require.
If you look here you can see that all arguments in __construct() are on the same line.

If I update the constructor to public function __construct(array $configuration, $plugin_id, $plugin_definition, \Drupal\flush_single_image\FlushSingleImage $flushSingleImageService) { I can run the migration locally. Is there a naming problem perhaps?

Here is a patch against 8.x-2.3 in case anyone else runs into this issue.

It's because of the class names for both the service and migrate plugin. They clash since their name is the same but are in different namespaces.

Thanks for the patch. I was working against latest dev branch. The fix or logic behind it should still apply.

Automatically closed - issue fixed for 2 weeks with no activity.