Issue #1430528 by mikeryan: Improve Document importance of migrating term parents...

diff --git a/CHANGELOG.txt b/CHANGELOG.txt
index 4f9c15d..411fb73 100644 (file)
--- a/CHANGELOG.txt
+++ b/CHANGELOG.txt
@@ -3,6 +3,7 @@ Migrate 2.3 RC1
 ===============
 
 Features and enhancements
+- #1430528 - Document importance of migrating term parents first.
 - #1418126 - Use node_delete_multiple on migrate-wipe.
 - #1424632 - Improve user migration performance by reducing hash count.
 - #1414616 - Automatically update migration arguments.

diff --git a/migrate_example/beer.inc b/migrate_example/beer.inc
index 0ea46aa..6249338 100644 (file)
--- a/migrate_example/beer.inc
+++ b/migrate_example/beer.inc
@@ -80,6 +80,9 @@ class BeerTermMigration extends BasicExampleMigration {
 
     // We are getting data from tables in the Drupal default database - first,
     // set up a query for this data.
+    // Note that by ordering on style_parent, we guarantee root terms are
+    // migrated first, so the parent_name mapping below will find that the
+    // parent exists.
     $query = db_select('migrate_example_beer_topic', 'met')
              ->fields('met', array('style', 'details', 'style_parent', 'region', 'hoppiness'))
              // This sort assures that parents are saved before children.