[CONTEDIT-57] FieldDefinitionKeyGenerator produces incomplete keys '.fieldName.label' for fields outside of a dialogs/tabs Created: 09/Mar/17 Updated: 20/Mar/17 Resolved: 20/Mar/17 |
|
| Status: | Closed |
| Project: | Content Editor |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.0 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Philip Mundt | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| 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
|
||||||||||||||||||||||||
| Epic Link: | Initial Article editor | ||||||||||||||||||||||||
| Sprint: | Basel 88 | ||||||||||||||||||||||||
| Story Points: | 3 | ||||||||||||||||||||||||
| Description |
|
When embedding a field outsize of an dialog/tab and i18nizing them, FieldDefinitionKeyGenerator will produce incomplete keys like .fieldName.label. This is somewhat expected but needs to be fixed as fields are used this way in the content editor. |
| Comments |
| Comment by Philip Mundt [ 20/Mar/17 ] |
|
Current solution produced a massive amount of log warnings: 2017-03-20 08:54:13,015 WARN .form.field.definition.FieldDefinitionKeyGenerator: Cannot obtain name of parent object: info.magnolia.editor.content.ContentDefinition$$EnhancerByCGLIB$$614a4d1b.getName() Instead of shunting the log and "hiding" the problem, re-introduction of the very simple KeyGenerator should solve the problem. Edit: This is the only problem, key generation otherwise works nicely. |
| Comment by Philip Mundt [ 20/Mar/17 ] |
|
Follow-up ticket was created |