[MGNLFORM-98] Text fields group has a non required 'group name' but without it the internal fields are not used Created: 15/Aug/11 Updated: 04/Nov/15 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Magnolia Form Module |
| Component/s: | field |
| Affects Version/s: | 1.2.3 |
| Fix Version/s: | 2.2.x |
| Type: | Bug | Priority: | Major |
| Reporter: | Tobias Mattsson | Assignee: | Unassigned |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | verify | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| 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: |
| Description |
|
The edit group field adds a set of edit fields within a group. In the dialog it asks for a group name which is indicated to be optional. The group name is in fact the controlName of the field. When binding the form will look at this paragraph and test for the controlName, if it's not there it will skip the paragraph, and all the fields it contains. If it's there it will be part of the form data with an empty value and its children will be bound as well. We should either, remove the requirement of having a controlName on this field. Or clearly indicate that it's required, although binding it to the form data serves no purpose. Note that the group name is also used as an id to the div containing the edit fields. Original report from user-list:
|
| Comments |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |