The migration doesn't currently take into the account language of host entity.

This might have been introduced with one of the patches I am rolling with below:

projects[field_collection][version] = 1.x-dev
projects[field_collection][subdir] = contrib
projects[field_collection][download][type] = git
projects[field_collection][download][revision] = ae778f2
projects[field_collection][download][branch] = 7.x-1.x
projects[field_collection][patch][1344672] = http://drupal.org/files/issues/field_collection-et-1344672-187.patch
projects[field_collection][patch][2075325] = http://drupal.org/files/issues/field_collection-field_collection_uuid-2075325-3.patch
CommentFileSizeAuthor
#2 migration_field_collection-2298877-02.patch806 bytesmarco.pagliarulo
#1 migration_field_collection-2298877-01.patch792 bytessylus
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sylus’s picture

sylus CreditAttribution: sylus commented
FileSize
migration_field_collection-2298877-01.patch792 bytes

Attaching patch.

marco.pagliarulo’s picture

marco.pagliarulo CreditAttribution: marco.pagliarulo as a volunteer commented
FileSize
migration_field_collection-2298877-02.patch806 bytes

In the patch #1 $entity->setHostEntity($this->hostEntityType) is wrong, the host entity is required. As attachment the patch fixed

sylus’s picture

sylus CreditAttribution: sylus commented
Status: Active » Reviewed & tested by the community

Oh your right I only ever set the language so didn't notice the other case was failing. Thanks for the catch and seems to work for both now!

  • marco.pagliarulo authored 2291aeb on 7.x-1.x 
    Issue #2298877 by sylus, marco.pagliarulo: Migration doesn't take into...
jmuzz’s picture

jmuzz CreditAttribution: jmuzz commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch.

Status: Fixed » Closed (fixed)

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