The d6_vocabulary_field_instance migration adds term reference fields to node types. Thus, the node types must already exist, but there's no dependency in the migration. Let's add it.

CommentFileSizeAuthor
#2 missing_dependency_in-2535364-2.patch1.29 KBmikeryan
d6_vocabulary_field_instance-dependency.patch528 bytesmikeryan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, d6_vocabulary_field_instance-dependency.patch, failed testing.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Murphysboro, IL, USA
CreditAttribution: mikeryan at Acquia commented
Status: Needs work » Needs review
FileSize
missing_dependency_in-2535364-2.patch1.29 KB

Needed to add the dependent id map in the test.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Needs review » Reviewed & tested by the community

Looks great.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed 5a9f6e7 and pushed to 8.0.x. Thanks!

  • alexpott committed 5a9f6e7 on 8.0.x 
    Issue #2535364 by mikeryan: Missing dependency in...

Status: Fixed » Closed (fixed)

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