search

SAP / openui5

OpenUI5 lets you build enterprise-ready web applications, responsive to all devices, running on almost any browser of your choice.
http://openui5.org
Apache License 2.0
2.96k stars 1.24k forks source link

Why ODataModel v4 isn't storing more then 1000 entries #3935

Closed CosminPascal closed 9 months ago

CosminPascal commented 10 months ago

We encountered a problem with ODataModel V4 in a sap.ui.table.Table. The table contains a list of products which we get via batch requests and we always request a little more then 50 entries but after we request approximately 1000 entries our table arrives at a state where the loading animation isn't disappearing because the dataRecevied event isn't triggered anymore. We used the dataRecevied event to stop the loading animation when the request was resolved. The weird thing is that the last request which is made is successful but the dataRecevied event isn't triggered.

We also tried to use the setSizeLimit with different values but it doesn't seem to have any impact on our ODataModel.

Our question is if this is a problem with our specific version of ui5 or if it's something else and if there a workaround for this.

We are using sapui5 version 1.96.0

boghyon commented 10 months ago

[...] the dataRecevied event isn't triggered anymore.

It's difficult to debug the issue without steps to reproduce it. But it might be already fixed with https://github.com/SAP/openui5/commit/40e573bddf952eeafae3930a07df9682953dbfb3 (also available for rel-1.96) and https://github.com/SAP/openui5/commit/b1c70a5bdf1103ee81a19c2c1fafb8c7fd737cf6. The latter change doesn't seem to be available for 1.96 yet.

Do you experience the missing dataReceived event only after scrolling down? Or up? Or both? 1.96.0 is outdated anyway. Please try with the latest patch release of e.g. 1.120 and, if the issue is still reproducible, share a minimal sample.

flovogt commented 9 months ago

There was no response from the author for 4 weeks, so I close this issue here. Feel free to re-open and provide further details, so we can have a look at the actual code.