[MGNLUI-7018] comboBox fails when dataProvider size > 500 Created: 19/Jan/22 Updated: 19/Jul/22 Resolved: 01/Apr/22 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | None |
| Affects Version/s: | 6.2.15 |
| Fix Version/s: | 6.2.18, 6.3.0 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Carlos Cantalapiedra | Assignee: | Quach Hao Thien |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | VN-Testing, nucleus | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | 3d 7h | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| 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
|
||||||||||||||||||||||||||||||||
| Documentation update required: |
Yes
|
||||||||||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||||||||||
| Epic Link: | Nucleus Quality Maintenance | ||||||||||||||||||||||||||||||||
| Sprint: | Nucleus 5, Nucleus 7 | ||||||||||||||||||||||||||||||||
| Story Points: | 2 | ||||||||||||||||||||||||||||||||
| Team: | |||||||||||||||||||||||||||||||||
| Description |
| Comments |
| Comment by Richard Gange [ 20/Jan/22 ] |
|
IMO we have to have some global way to change the comboBox field max. Otherwise it could be a tedious issue of having to reconfigure a bunch of fields that were working before the limit was introduced. |
| Comment by Roman Kovařík [ 15/Feb/22 ] |
|
Moving back to discovery, see https://git.magnolia-cms.com/projects/PLATFORM/repos/ui/pull-requests/1624/overview?commentId=92461. |
| Comment by Quach Hao Thien [ 18/Feb/22 ] |
DiscoverySteps to reproduce
session = ctx.getJCRSession('dam') session.getRootNode().addNode('foo', 'mgnl:folder') folderNode = session.getNode('/foo') for(int i in 1..1000) { println(i) folderNode.addNode(i as String, "mgnl:asset") } session.save()
form: properties: headlineLevel: $type: comboBoxField defaultValue: small datasource: $type: optionListDatasource options: - name: big value: big - name: medium value: medium - name: small value: small headline: $type: textField i18n: true text: class: info.magnolia.dam.app.field.DamRichTextFieldDefinition i18n: true image: $type: damLinkField imageComboBox: $type: comboBoxField datasource: $type: jcrDatasource workspace: dam rootPath: /foo imagePosition: $type: comboBoxField defaultValue: below datasource: $type: optionListDatasource options: - name: above value: above - name: below value: below imageAltText: $type: textField i18n: true imageTitle: $type: textField i18n: true imageCaption: $type: textField i18n: true imageCredit: $type: textField i18n: true layout: $type: tabbedLayout tabs: - name: tabText fields: - name: headlineLevel - name: headline - name: text - name: tabImage fields: - name: image - name: imageComboBox - name: imagePosition - name: imageAltText - name: imageTitle - name: imageCaption - name: imageCredit
Expected resultsAll assets are displayed Actual resultsIllegalStateException is thrown (Client tried fetch more rows than allowed. This is denied to prevent denial of service.) WorkaroundSet the pageLength to 500 (or less) in the combobox definition <- this doesn't fix the issue Discovery notes
Development notes
|
| Comment by Richard Gange [ 18/Feb/22 ] |
|
|
| Comment by Roman Kovařík [ 14/Mar/22 ] |
|
Solution:
|
| Comment by Roman Kovařík [ 23/Mar/22 ] |
|
Reopened: we can still run into the issue if we set pageLength to 0. |
| Comment by Ashraf Khamis [ 31/Mar/22 ] |
|
Doc update: https://git.magnolia-cms.com/projects/DOCUMENTATION/repos/product-docs/pull-requests/968/overview |
| Comment by Quach Hao Thien [ 31/Mar/22 ] |
|
for RN:
|