Problem/Motivation

Webform Migration fails silently when it cannot migrate a field type.

Steps to reproduce

  • Enable the webform_reference module in D7
  • Create a webform in D7 that has an entity reference field
  • use the webform_migrate module to migrate that webform

Proposed resolution

Update the `default:` in D7Webform::buildFormElements() to output informational text.

Remaining tasks

Update the `default:` in D7Webform::buildFormElements() to output informational text.

CommentFileSizeAuthor
#2 field-failure-output-3202572-2.patch601 bytessrjosh

Comments

srjosh created an issue. See original summary.

srjosh’s picture

srjosh
he/him
commented
Title: Output information on un-migrated field types. » Output information on un-migrated field types
StatusFileSize
new field-failure-output-3202572-2.patch601 bytes

Patch attached!

srjosh’s picture

srjosh
he/him
commented
Status: Active » Needs review
andriyun’s picture

andriyun commented
Status: Needs review » Needs work 
+++ b/src/Plugin/migrate/source/d7/D7Webform.php
@@ -495,7 +495,7 @@ class D7Webform extends DrupalSqlBase implements ImportAwareInterface, RollbackA
+          echo 'Field ' . $element['name'] . " of type " . $element['type'] . " was unable to be migrated on form " . $nid . "\n";

Plugin `src/Plugin/migrate/source/d7/D7Webform.php` has been implemented in a way that is write yml markup through `echo` function.
Changes proposed in patch will make this markup invalid.

If you like to write some debug message during import, another channels should be used for that.