[ABTEST-539] VariantJcrDataProvider relies on implementation of JcrDataProvider#size Created: 25/Jan/23 Updated: 01/Feb/23 |
|
| Status: | Open |
| Project: | A/B Testing |
| Component/s: | None |
| Affects Version/s: | 1.0.1 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Roman Kovařík | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| 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
|
||||||||
| Epic Link: | AuthorX Maintenance | ||||||||
| Team: | |||||||||
| Description |
Steps to reproduce
Expected resultsI can still work with the UI Actual resultsInfinite loading bar, browser tab not responsive. WorkaroundOpen new browser tab to unlock the UI, the Variants tab still can't be used though. Development notesWe can see in https://git.magnolia-cms.com/projects/ADDON/repos/ab-testing/browse/magnolia-abtesting-ui/src/main/java/info/magnolia/abtesting/app/dataprovider/VariantJcrDataProvider.java#60 that it overrides #fetch but not #size, relying of the fact the #fetch is called to compute super#size. Suggested solution: Implement #size instead of relying on certain super impl. |