Problem/Motivation

Field 'description' for taxonomy term has wrong export (without format).

Steps to reproduce

Export taxonomy term with not empty description.

Proposed resolution

Add 'format: basic_html' for field 'desctription'.

CommentFileSizeAuthor
#7 single_content_sync-3439477-add-format-option-7.patch1.67 KBvasyapledov
#3 single_content_sync-3439477-add-format-option-3.patch1.66 KBvasyapledov
#2 single_content_sync-3439477-add-format-option-2.patch1.46 KBvasyapledov

Issue fork single_content_sync-3439477

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

vasyapledov created an issue. See original summary.

vasyapledov’s picture

vasyapledov commented
StatusFileSize
new single_content_sync-3439477-add-format-option-2.patch1.46 KB

Fixed. Need to check.

vasyapledov’s picture

vasyapledov commented
StatusFileSize
new single_content_sync-3439477-add-format-option-3.patch1.66 KB

New variant.

vasyapledov’s picture

vasyapledov commented
vasyapledov’s picture

vasyapledov commented
Assigned: vasyapledov » Unassigned
Status: Needs work » Needs review
nginex’s picture

nginex commented
Status: Needs review » Needs work

You need to use $entity->getDescription() as a value and $entity->getFormat() as a format, currently the format is hardcoded on your patch

vasyapledov’s picture

vasyapledov commented
Status: Needs work » Needs review
StatusFileSize
new single_content_sync-3439477-add-format-option-7.patch1.67 KB

Yes, you are right. Fixed.

nginex’s picture

nginex commented
Issue tags: +LutskGCW25

kazuko.murata made their first commit to this issue’s fork.

kazuko.murata opened merge request !180

kazuko.murata’s picture

kazuko.murata commented

Rerolled #7 and created MR.

jannakha’s picture

jannakha commented
Version: 1.4.4 » 1.4.x-dev
Status: Needs review » Reviewed & tested by the community

thank you for your contribution!
MR works as described:

before patch:

uuid: 33a0883a-77b9-4677-869c-35b6fb121e5a
entity_type: taxonomy_term
bundle: location_types
base_fields:
  name: Branch/Office
  weight: 0
  langcode: en
  description: '<p>test</p>'
  parent: 0
  url: null
custom_fields: {  }

after patch:

uuid: 33a0883a-77b9-4677-869c-35b6fb121e5a
entity_type: taxonomy_term
bundle: location_types
base_fields:
  name: Branch/Office
  weight: 0
  langcode: en
  description:
    value: '<p>test</p>'
    format: basic_html
  parent: 0
  url: null
custom_fields: {  }
nginex’s picture

nginex commented
Status: Reviewed & tested by the community » Fixed

Thanks for the patch

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

  • nginex committed 56ccc355 on 1.4.x authored by kazuko.murata 
    feat: #3439477 Add 'format' option for 'description' taxonomy term base...

Status: Fixed » Closed (fixed)

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