[MGNLUI-5260] multi-reference properties are broken - storing paths instead of UUID Created: 19/Jun/19 Updated: 21/Jun/19 Resolved: 20/Jun/19 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | 6.1 |
| Fix Version/s: | 6.1 |
| Type: | Bug | Priority: | Critical |
| Reporter: | Christoph Meier | Assignee: | Mikaël Geljić |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | 0.75d | ||
| 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: | |||||||||||||||||
| Sprint: | 6.1 Release Sprint | ||||||||||||||||
| Description |
|
When a CT has a reference property to another property, and it is marked as "multiple", then the generated app does not correctly configure the identifierToPathConverter node. Thiis causes links to be stored as paths instead of UUID's, which will cause problems in templating and other places. When the CT property is not multiple, then the identifierToPathConverter is created correctly in the app. — Given we have a CT with a CT-based app,
# ...
model:
nodeType: mt:tourOffice
properties:
- name: tourGuides
type: reference:tourGuide
multiple: true
# ...
Then the definition app claims a MINOR PROBLEM Definition: tourOffices-app Origin: /content-type-examples/apps/tourOffices-app.yaml Details: Property [identifierToPathConverter] is set to null in definition. Location: tourOffices-app/subApps/detail/editor/form/tabs/default/{name=name, i18n=false, required=true, type=String, fieldType=text}/tourGuides/tourGuides/identifierToPathConverter Type: Source data processing problem Severity: MINOR When removing the multiple: true, there is no more problem. This is a regression. The given example has worked. (GIT: content-type-examples). |
| Comments |
| Comment by Christopher Zimmermann [ 19/Jun/19 ] |
|
Storing with path instead of UUID is a big problem. Is it just with multiple, or also single references? |
| Comment by Christoph Meier [ 19/Jun/19 ] |
|
czimmermann - it happens only in the "multiple" case. |
| Comment by Dai Ha [ 21/Jun/19 ] |
|
Verified on magnolia-dx-core-demo-webapp-6.1-20190620.142559-77.war. There is no problem in definition app and references saved as UUID. |