Problem/Motivation

Our lovely migration system has no support for migrating the incredibly important and widely-used node reference fields, and their values, from Drupal 6 to Drupal 8. It's crucial for us to support this.

Proposed resolution

Add a field plugin to handle Drupal 6 nodereference fields.

Remaining tasks

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#7 2814949-7.patch2.16 KBphenaproxima

Comments

phenaproxima created an issue. See original summary.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Issue summary: View changes
Status: Active » Postponed

Postponing on #2447727: Add base class for migrating reference fields.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.0-alpha1 will be released the week of January 30, 2017, which means new developments and disruptive changes should now be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Postponed » Closed (duplicate)

D6 node reference field definitions are now supported as of #2872660: Migrate D6 node reference fields to D8 entity reference field definitions, which landed today. Closing as duplicate!

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Issue summary: View changes
Status: Closed (duplicate) » Active
Related issues: +#2872660: Migrate D6 node reference fields to D8 entity reference field definitions

Actually, no -- reopening because that issue only migrates the field definitions, but not the values. This issue can be where we migrate the actual field values. Updating the IS.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Title: Support Drupal 6 node reference fields » Support migration of node reference field values from Drupal 6
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 2814949-7.patch2.16 KB

A patch! Finally, a patch. With the field definitions having been handled over in #2872660: Migrate D6 node reference fields to D8 entity reference field definitions, this part was easy as pie. Let's get this done.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +8.4.0 release notes

Should this be added to release notes? Tagging. Reviewiewed the code and marking RTBC. We now have node reference fully functional. Only user reference left to make non-i18n from D6 fully, 100% feature complete!

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented 
+++ b/core/modules/migrate_drupal/src/Plugin/migrate/field/NodeReference.php
@@ -0,0 +1,43 @@
+}
\ No newline at end of file

Missing newline

  • catch committed 12ac3a6 on 8.4.x 
    Issue #2814949 by phenaproxima: Support migration of node reference...

  • catch committed 3d2a521 on 8.3.x 
    Issue #2814949 by phenaproxima: Support migration of node reference...
catch’s picture

catch
he/him
Primary language English
commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Really great to see this one green.

Committed/pushed to 8.4.x, and cherry-picked to 8.3.x, thanks!

Agreed on release notes mention.

Status: Fixed » Closed (fixed)

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

vpa24’s picture

vpa24 commented

It is still doesn't work for me. This is my code:

  field_node:
    -
      plugin: iterator
      source: field_node
      process:
        target_id:
          plugin: migration_lookup
          migration:
            - upgrade_d6_node_page
          source: nid

In drupal 8, after migration I don't see my cck node reference field.