[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: |
|
||||||||
| Issue Links: |
|
||||||||
| 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:
|
| 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 |