Problem/Motivation

While writing tests for the Drupal 7-to-8 migration path, I've run into a lot of config schema exceptions due to migrations defining source constants. Turns out that migrate's data_types.schema.yml doesn't define the constants mapping as part of the migrate_source data type; adding it solves this problem.

This is a major issue because it blocks almost all child issues of #2456259: [META] Drupal 7 to Drupal 8 Migration path.

Proposed Resolution

Add the constants mapping as an ignored property of the migrate_source data type.

Remaining Tasks

Patch, review, commit.

API/UI Changes

Nil.

CommentFileSizeAuthor
#2 2495851-2.patch597 bytesphenaproxima
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

phenaproxima’s picture

Issue summary: View changes
phenaproxima’s picture

Status: Active » Needs review
FileSize
597 bytes
YesCT’s picture

Issue tags: +Configuration schema
benjy’s picture

Status: Needs review » Needs work

Sources need to define their constants in the schema using migrate_entity_constant.

phenaproxima’s picture

Status: Needs work » Closed (won't fix)

@benjy explained the issue, and its real fix, to me on IRC. This patch is the sledgehammer approach.