My thought here is that there are situations where I would like to merge two fields that are coming over as arrays. Particularly in the case of dealing with entity reference fields, or other fields that have multiple values. An example migration may be like the following:

  source:
    plugin: d7_node
  process:
    temp_body:
      plugin: iterator
      source: field_section
      process:
        target_id:
          plugin: migration
          migration: field_collection_field_section_to_paragraph
          source: value
    temp_images:
      plugin: iterator
      source: field_image
      process
        target_id:
          plugin: migration
          migration: image_entities_to_paragraph
          source: fid
    paragraphs_field:
      plugin: merge
      source:
        - @temp_body
        - @temp_images
  destination:
    plugin: 'entity:node'

In this case I'm able to merge entities created from 2 different migrations into one entity reference field (specifically paragraphs). You might even be able to do this with 2 different types of paragraphs created using the EntityGenerate Process Plugin.

CommentFileSizeAuthor
#5 interdiff-2747939-3-5.txt718 bytesgeneralredneck
#5 migrate_plus-merge_process_plugin-2747939-5.patch2.1 KBgeneralredneck
#3 migrate_plus-merge_process_plugin-2747939-3.patch2.15 KBgeneralredneck
#2 migrate_plus-merge_process_plugin-2747939-2.patch1000 bytesgeneralredneck
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

generalredneck created an issue. See original summary.

generalredneck’s picture

generalredneck
him/his
Primary language English
Location Texas, USA 🇺🇸
CreditAttribution: generalredneck commented
Status: Active » Needs review
FileSize
migrate_plus-merge_process_plugin-2747939-2.patch1000 bytes

Lets try this on for size

generalredneck’s picture

generalredneck
him/his
Primary language English
Location Texas, USA 🇺🇸
CreditAttribution: generalredneck commented
FileSize
migrate_plus-merge_process_plugin-2747939-3.patch2.15 KB

We actually need merge not Union like my first attempt was

FluxSauce’s picture

FluxSauce CreditAttribution: FluxSauce at Four Kitchens commented
Status: Needs review » Needs work

Seems to function, but includes two var_dumps.

drush mi NAMEOFMIGRATION
array(2) {
  [0]=>
  array(3) {
    [0]=>
    array(2) {
      ["target_id"]=>
      string(3) "127"
      ["target_revision_id"]=>
      string(3) "127"
    }
    [1]=>
    array(2) {
      ["target_id"]=>
      string(3) "128"
      ["target_revision_id"]=>
      string(3) "128"
    }
generalredneck’s picture

generalredneck
him/his
Primary language English
Location Texas, USA 🇺🇸
CreditAttribution: generalredneck commented
Status: Needs work » Needs review
FileSize
migrate_plus-merge_process_plugin-2747939-5.patch2.1 KB
interdiff-2747939-3-5.txt718 bytes

FAILURE!

Ok, so here's the debug free version.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan as a volunteer and at Virtuoso Performance commented
Status: Needs review » Fixed

Committed, thanks!

Jaesin’s picture

Jaesin CreditAttribution: Jaesin at Chapter Three commented

The Merge plugin throws MigrateException when source data isn't as nice as it could be.

In general, migrate handles source input pretty well as far as dealing with a single value vs an array. 

<?php
      if (!is_array($item)) {
        throw new MigrateException('One of the items is not an array that can be merged.');
      }
      $new_value = array_merge($new_value, $item);
?>

Could be replaced with: 

<?php
    foreach($value as $item) {
      $new_value = array_merge($new_value, (array) $item);
    }
?>

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.