https://www.drupal.org/project/link_class

Comments

Grimreaper created an issue. See original summary.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Assigned: grimreaper » Unassigned

Will reassign to myself when I will have time to test.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Title: Test compability with Link class » Support link fields options
Category: Task » Feature request

Tested with https://www.drupal.org/project/link_class but it will be the problem with https://www.drupal.org/project/link_attributes and other modules that use this options field.

The field is empty in JSONAPI.

Opened an issue in the JSON API issue queue: #2911477: Link fields' "options" not normalized.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Related issues: +#2911477: Link fields' "options" not normalized, +#2895532: @DataType=map cannot be normalized, affects @FieldType=link, @FieldType=map

See #2911477-3: Link fields' "options" not normalized. The root cause is being fixed in #2895532: @DataType=map cannot be normalized, affects @FieldType=link, @FieldType=map.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Status: Active » Postponed

Thanks Wim Leers.

I have just tested the patch from the comment #2895532-96: @DataType=map cannot be normalized, affects @FieldType=link, @FieldType=map and it works ok. I see the options in the JSON and its are transmitted using Entity share.

grimreaper’s picture

grimreaper
Primary language French
Location France 🇫🇷
commented
Status: Postponed » Fixed

Hello,

I have made a test on:

  • Drupal core: 8.6.0
  • Entity share: 8.x-1.0-alpha9
  • Link attributes: 8.x-1.3
  • Link class: 8.x-1.3

The link attributes are now serialized :D, no more patch or workaround needed.

Marking the issue as fixed. \o/

Status: Fixed » Closed (fixed)

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