See Drupal core issue for motivation and proposed resolution: #3247950: Throw consistent exceptions on invalid process plugin configurations. Let's support this initiative and apply the same to Migrate Plus process plugins. We can probably be more relaxed and think less about BC changes, but we'll see how it goes.
1) Review available process plugins and make sure they throw consistent exceptions;
2) If there is plugin to be fixed, create a child issue;
3) Submit a merge request with proposed changes;
Review existing process plugins and create child issues.
Comments
Comment #2
matroskeenComment #3
matroskeenWe still have some inconsistency in EntityLookup.php, but this plugin is trying to guess missing properties in
::transformmethod, so not sure if we can do something about it.It looks like we also have some usages of
InvalidPluginDefinitionExceptioninstead ofInvalidArgumentException.Let's wait for #3247950: Throw consistent exceptions on invalid process plugin configurations and then do the rest.