Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 291721

MigrateSkipRowException should document that it is for use within migration process plugins

$
0
0

Problem/Motivation

Looking at MigrateSkipRowException, you might think you can throw it anywhere in a migration.

However, for instance, throwing it in a destination plugin causes an error. It looks like it's caught in:

- processing a row
- invoking migrate_prepare_row() hooks
- MigrationLookup -- which might be because that involves processing a row?

Steps to reproduce

Proposed resolution

Add documentation to the exception class docblock.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet


Viewing all articles
Browse latest Browse all 291721

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>