Problem/Motivation

When migrating taxonomy terms from D7 to D8 the description value field is imported correctly, but not the description format.

Proposed resolution

Correct the omission in the d7_taxonomy_term.yml migration template.

Remaining tasks

Test and deploy

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 d7_taxonomy_description_format-2691641-4.patch3.67 KBjofitz
#4 interdiff.txt3.12 KBjofitz
#2 d7_taxonomy_description_format-2691641-2.patch554 bytesjofitz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Jo Fitzgerald created an issue. See original summary.

jofitz’s picture

jofitz CreditAttribution: jofitz commented
Issue summary: View changes
Status: Active » Needs review
Issue tags: +migrate-d7-d8
FileSize
d7_taxonomy_description_format-2691641-2.patch554 bytes

Provide suggested patch.

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Needs review » Needs work
Issue tags: +Needs tests
jofitz’s picture

jofitz CreditAttribution: jofitz at ComputerMinds commented
Status: Needs work » Needs review
Issue tags: -Needs tests
FileSize
interdiff.txt3.12 KB
d7_taxonomy_description_format-2691641-4.patch3.67 KB

Added tests.

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Needs review » Reviewed & tested by the community

Thanks, looks good.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 4: d7_taxonomy_description_format-2691641-4.patch, failed testing.

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Needs work » Reviewed & tested by the community

I think the failure could have been in head.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.2.x and cherry-picked to 8.1.x. Thanks!

  • catch committed 327f8e3 on 8.2.x 
    Issue #2691641 by Jo Fitzgerald, benjy: D7 taxonomy description format...

  • catch committed 727bead on 8.1.x 
    Issue #2691641 by Jo Fitzgerald, benjy: D7 taxonomy description format...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed
james.williams’s picture

james.williams CreditAttribution: james.williams at ComputerMinds commented

Status: Fixed » Closed (fixed)

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

jofitz’s picture

jofitz CreditAttribution: jofitz at ComputerMinds commented