- 🇦🇺Australia acbramley
This needs a reroll into an MR against 11.x. Would also be good to get the IS updated with steps to reproduce.
The \Drupal\migrate\Plugin\migrate\id_map\Sql
class assumes that the migration message table exists if the related map table exists.
This is OK in most cases, but in the context of
✨
Allow overriding id map migrate plugin
Needs review
, and using a custom id map plugin that allows overriding the map/message tables; the assumption does not hold up.
Remove the assumption, this can be even be only for correctness :-)
N.A.
N.A.
N.A.
Needs work
11.0 🔥
migration system
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
The patch will have to be re-rolled with new suggestions/changes described in the comments in the issue.
Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This needs a reroll into an MR against 11.x. Would also be good to get the IS updated with steps to reproduce.