[MGNLUI-3872] Multivalue field with composite field can not be set as required Created: 02/May/16  Updated: 19/Feb/21  Resolved: 19/Feb/21

Status: Closed
Project: Magnolia UI
Component/s: forms
Affects Version/s: 5.4.6
Fix Version/s: None

Type: Bug Priority: Blocker
Reporter: Ervin Vystup Assignee: Unassigned
Resolution: Won't Fix Votes: 2
Labels: support
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Attachments: PNG File save_changes-with_text.png     PNG File save_changes.png     XML File test.xml    
Issue Links:
relation
is related to CONTEDIT-202 Provide solution for validation UI wi... Closed
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:
Story Points: 8

 Description   

Setting required=true in multivalue field with nested composite field causes validation to fail despite added items.

To reproduce:

  1. import test.xml to /modules/tours/apps/tours/subApps/detail/editor/form/tabs/tour/fields/
  2. go to Tours app -> choose some tour -> "Edit tour" -> "Save changes"
  3. click on "Add" -> insert some text -> "Save changes"


 Comments   
Comment by Marty Glaubitz [ 01/Oct/18 ]

We're also dealing with this. Using Magnolia 5.7

Comment by Ervin Vystup [ 03/Oct/18 ]

Hi martyglaubitz,

I have found out today that in the mentioned test case the issue is connected with the StaticField used. Without this field, the "required" property doesn't cause any issue. Could you, therefore, attach here your configuration for me to see your use case please? 

Thank you, Ervín

Comment by Marty Glaubitz [ 04/Oct/18 ]

@evystup no - we're not using static fields in our configuration. Here it is:

label: fields.richText
layout: vertical
fieldType: composite
fields:
  - name: content
    fieldType: richText
    label: fields.text
transformerClass: com.our.project.ChildNodeCompositeTransformer
Comment by Šimon Demočko [ 19/Feb/21 ]

we won't fix it for this version, it has reached EOL. The current ticket is MGNLUI-6563

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