[MAGNOLIA-6676] Relocate Map2BeanTransformer to the core Created: 20/May/16 Updated: 09/Feb/17 Resolved: 24/May/16 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | None |
| Affects Version/s: | 5.4.6 |
| Fix Version/s: | 5.4.7, 5.5 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Aleksandr Pchelintcev | Assignee: | Aleksandr Pchelintcev |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | koiinos | ||
| Remaining Estimate: | 0d | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | 0d | ||
| 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: | Basel 45 | ||||||||||||||||
| Story Points: | 0 | ||||||||||||||||
| Description |
|
Currently M2B resides in the magnolia-configuration module and => is not available for the classes in magnolia-core. Since M2B is required now for resolution of light module descriptors which is not directly related to configuration and registries, we have to relocate the component in a backwards-compatible fashion. NOTE: the Map2BeanTransformerTest has not yet been relocated (since it depends on the test dummy pojos in magnolia-config and lot's of other tests depend on them), I propose to relocate those tests and dummy pojos during proper MAGNOLIA-6295 effort. |
| Comments |
| Comment by Jan Haderka [ 20/May/16 ] |
|
since we want to split core, I don't see much of an advantage in actually adding to it prior to the split. But of course I understand why you need it in relation to module descriptors being resolved from FS. |
| Comment by Aleksandr Pchelintcev [ 24/May/16 ] |
|
This issue is tackled during implementation of |