[MAGNOLIA-8368] Class should take precedence over $type Created: 04/Mar/22 Updated: 03/Jun/22 Resolved: 29/Apr/22 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.2.19 |
| Type: | Bug | Priority: | Medium |
| Reporter: | Tim Molenaar | Assignee: | Roman Kovařík |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | nucleus | ||
| Σ Remaining Estimate: | Not Specified | Remaining Estimate: | Not Specified |
| Σ Time Spent: | 5h | Time Spent: | 0.25d |
| Σ Original Estimate: | Not Specified | Original Estimate: | Not Specified |
| Environment: |
magnolia-personalization-pages |
||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||
| Sub-Tasks: |
|
||||||||||||||||||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||||||||||||||||||
| Task DoD: |
[X]*
Doc/release notes changes? Comment present?
[X]*
Downstream builds green?
[X]*
Solution information and context easily available?
[X]*
Tests
[X]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||||||||||||||||||
| Bug DoR: |
[X]*
Steps to reproduce, expected, and actual results filled
[X]*
Affected version filled
|
||||||||||||||||||||||||||||||||||||||||
| Release notes required: |
Yes
|
||||||||||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||||||||||
| Epic Link: | Support | ||||||||||||||||||||||||||||||||||||||||
| Sprint: | Nucleus 9 | ||||||||||||||||||||||||||||||||||||||||
| Story Points: | 3 | ||||||||||||||||||||||||||||||||||||||||
| Team: | |||||||||||||||||||||||||||||||||||||||||
| Description |
| Comments |
| Comment by Roman Kovařík [ 21/Mar/22 ] |
|
ajones The ticket was added to Nucleus board, does the team need a help or the label was added by accident? |
| Comment by Dominik Maslanka [ 23/Mar/22 ] |
|
during investigation was found out that this is Nucleus domain. |
| Comment by Rabie Hayoun [ 22/Apr/22 ] |
|
This need to be fixed at annotation processing time: https://git.magnolia-cms.com/projects/PLATFORM/repos/main/browse/magnolia-configuration/src/main/java/info/magnolia/config/resolver/AnnotationProcessingTypeResolver.java |
| Comment by Roman Kovařík [ 26/Apr/22 ] |
|
class property in YAML config now takes precedence over $type. Reasoning / typical use case:
This might affect configurations where a custom $type would override a class. Please adjust your configuration in that case and A] use class instead of type in the decoration file B] or if the decorated definition is your own, change it to use a $type. |