[MAGNOLIA-9198] Refactor ProviderImpl to gain access to the RepositoryContext Created: 30/Nov/23 Updated: 21/Dec/23 Resolved: 15/Dec/23 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.3.0 |
| Type: | Task | Priority: | Neutral |
| Reporter: | Michael Duerig | Assignee: | Michael Duerig |
| Resolution: | Done | Votes: | 0 |
| Labels: | dx-core-6.3 | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||
| Template: |
|
||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||
| Task DoR: |
Empty
|
||||||||||||||||||||
| Release notes required: |
Yes
|
||||||||||||||||||||
| Documentation update required: |
Yes
|
||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||
| Epic Link: | Collect Usage Metrics | ||||||||||||||||||||
| Work Started: | |||||||||||||||||||||
| Approved: |
Yes
|
||||||||||||||||||||
| Description |
|
As discovered in Implementation notesThere's currently two proposals on how this could be implemented, both with their own pros and cons:
1. is less risky as it does not touch the existing design and implementation much but it introduces some technical debt as we need to copy and slightly change a few classes from Jackrabbit (BindableMgnlRepository, BindableRepositoryFactory, RegistryHelper). 2. is more risky and reveals some tests that need fixing. There is a potential ripple effect of this causing similar test failures in other modules. Conceptually the approach is cleaner and introduces less technical debt. |
| Comments |
| Comment by Mikaël Geljić [ 06/Dec/23 ] |