[MGNLUI-5521] i18n not working in multi-value composite field Created: 22/Nov/19  Updated: 29/Sep/22  Resolved: 19/Aug/21

Status: Closed
Project: Magnolia UI
Component/s: None
Affects Version/s: 5.6.11, 5.7.5
Fix Version/s: None

Type: Bug Priority: Neutral
Reporter: Viet Nguyen Assignee: Unassigned
Resolution: Outdated Votes: 2
Labels: maintenance, tech-debt, y2k
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: File workaround-dialog.yaml    
Issue Links:
causality
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: I18n fields issues

 Description   

From SUPPORT-10623: i18n not working in multi-value composite field
Use below simplified version of the ticket, we can verify the issue of the i18n tab (under form > tabs) configuration:

    - name: tabReferenceAvis
      label: Reference Avis
      i18n: true
      fields:
        - name: referenceTitle
          class: info.magnolia.ui.form.field.definition.TextFieldDefinition
          i18n: true
          label: Title
        - name: referenceList
          class: info.magnolia.ui.form.field.definition.MultiValueFieldDefinition
          i18n: true
          label: Items
          transformerClass: info.magnolia.ui.form.field.transformer.multi.MultiValueSubChildrenNodePropertiesTransformer
          field:
            class: info.magnolia.ui.form.field.definition.CompositeFieldDefinition
            i18n: true
            transformerClass: info.magnolia.ui.form.field.transformer.composite.NoOpCompositeTransformer
            fields:
              refHeading:
                class: info.magnolia.ui.form.field.definition.TextFieldDefinition
                i18n: true
                label: Heading
              refValue:
                class: info.magnolia.ui.form.field.definition.TextFieldDefinition
                i18n: true
                label: Value
              refLink:
                appName: pages
                class: info.magnolia.ui.form.field.definition.LinkFieldDefinition
                description: Will render a link when Page type is selected
                i18n: true
                label: Value URL
                targetWorkspace: website
              refValueNote:
                class: info.magnolia.ui.form.field.definition.TextFieldDefinition
                i18n: true
                label: Value note
              refType:
                class: info.magnolia.ui.form.field.definition.SelectFieldDefinition
                label: Value type
                options:
                  1:
                    label: Page
                    selected: true
                  2:
                    label: Telephone
                  3:
                    label: Email
              refFeature:
                class: info.magnolia.ui.form.field.definition.SelectFieldDefinition
                label: Feature value
                options:
                  1:
                    label: No
                    selected: true
                  2:
                    label: Yes

Changing language does not affect to configured sub-fields.

PS: please also consider fixing for "workaround doesn't allow for the multivalue field to be moved and swapped with other list items"



 Comments   
Comment by Viet Nguyen [ 13/Dec/19 ]

Workaround
by replacing info.magnolia.ui.form.field.transformer.multi.MultiValueSubChildrenNodePropertiesTransformer with info.magnolia.ui.form.field.transformer.multi.DelegatingMultiValueFieldTransformer
And replace info.magnolia.ui.form.field.transformer.composite.NoOpCompositeTransformer with info.magnolia.ui.form.field.transformer.composite.DelegatingCompositeFieldTransformer
Reference code:
workaround-dialog.yaml

Comment by otto scholten [ 07/Apr/20 ]

However, viet your workaround doesn't allow for the multivalue field to be moved and swapped with other list items?

Comment by Viet Nguyen [ 07/Apr/20 ]

Thanks otto for your information, we'd also consider fixing your reported issue when implementing requested function.

Generated at Mon Feb 12 09:27:23 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.