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

CommentFileSizeAuthor
#3 2686651-3.patch763 bytesquietone
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

quietone created an issue. See original summary.

quietone’s picture

These tests use the test_page content type; MigrateNodeTypeTest.php, MigrateFieldFormatterSettingsTest.php, MigrateNodeBuilderTest.php.

quietone’s picture

Status: Active » Needs review
FileSize
763 bytes

Truncating content_type_page clears up the conflicts, and works on the three tests mentioned in #2. I wonder if more tests are effected.

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

benjy’s picture

Status: Needs review » Reviewed & tested by the community

RTBC if it still applies.

  • catch committed eb192d8 on 8.2.x
    Issue #2686651 by quietone: Same nid in two content types in d6_dump
    
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.1.x and 8.2.x, thanks!

  • catch committed 76de5c4 on 8.1.x
    Issue #2686651 by quietone: Same nid in two content types in d6_dump
    
    (...

Status: Fixed » Closed (fixed)

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