Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
While working on link migration tests I spotted what I believe are errors in d6_dump. Nid 1 in two content type tables, content_type_page and content_type_story. Also vid 3 corresponds to different nids in those tables. And vid 5 is funny too. In node_revisions it goes with nid 1 but in content_type_story it goes with nid 2.
MariaDB [d6_dump]> select nid,vid from content_type_story;
+-----+-----+
| nid | vid |
+-----+-----+
| 1 | 1 |
| 1 | 2 |
| 2 | 3 |
| 2 | 5 |
| 9 | 12 |
+-----+-----+
5 rows in set (0.00 sec)
MariaDB [d6_dump]> select nid,vid from content_type_page;
+-----+-----+
| nid | vid |
+-----+-----+
| 1 | 1 |
| 1 | 3 |
+-----+-----+
2 rows in set (0.03 sec)
MariaDB [d6_dump]> select nid, vid from node_revisions;
+-----+-----+
| nid | vid |
+-----+-----+
| 1 | 1 |
| 1 | 2 |
| 2 | 3 |
| 3 | 4 |
| 1 | 5 |
| 4 | 6 |
| 5 | 7 |
| 6 | 8 |
| 7 | 9 |
| 8 | 10 |
| 9 | 11 |
| 9 | 12 |
Proposed resolution
TBD
Remaining tasks
Fix the dump
Determine if this affects any tests.
User interface changes
N/A
API changes
N/A
Data model changes
N/A
Comment | File | Size | Author |
---|---|---|---|
#3 | 2686651-3.patch | 763 bytes | quietone |
Comments
Comment #2
quietone CreditAttribution: quietone as a volunteer commentedThese tests use the test_page content type; MigrateNodeTypeTest.php, MigrateFieldFormatterSettingsTest.php, MigrateNodeBuilderTest.php.
Comment #3
quietone CreditAttribution: quietone as a volunteer commentedTruncating content_type_page clears up the conflicts, and works on the three tests mentioned in #2. I wonder if more tests are effected.
Comment #5
benjy CreditAttribution: benjy at PreviousNext commentedRTBC if it still applies.
Comment #7
catchCommitted/pushed to 8.1.x and 8.2.x, thanks!