Openbravo Issue Tracking System - Retail Modules
View Issue Details
0047575Retail ModulesWeb POSpublic2021-08-09 12:492021-08-23 06:11
cberner 
ranjith_qualiantech_com 
highmajorhave not tried
closedfixed 
5
 
RR21Q2.2RR21Q2.2 
Production - Confirmed Stable
2020-02-28
main
https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/87c3e6d251f0c8db080607ba3c4739a551bde0b9 [^]
No
0047575: Incremental Masterdata refresh only loads the first batch of each model, ignoring those in the 2nd+ batch
If an incremental masterdata refresh is done and there is a model, for example Product with new entries higher than masterdata refresh batch size(this is by default 10000, but it can be modified with OBMOBC_MasterdataBatchSize preference) it will only load the first batch of products, ignoring all the rest.

Same applies to all masterdata models.
Previously log at least once into WebPOS so it completely refreshes masterdata.

1. Login in Backoffice as White Valley Group admin
2. Go to preferences and create a new one:
  - Property: Masterdata models batch size
  - Value: 2
  - Make sure visibility at User is none (seems Openbravo by default)
3. Go to Color window and create at least 3 new colors on the default color palette, or on a new one.
4. Login in WebPOS and click on top-right gear, Refresh Masterdata
5. Using dev-tools, go to Application -> IndexedDB -> Color, check that not all elements have been loaded. Checking console you should be able to see this:

[incrementalMasterdataRefresh] Starting incremental RefreshMasterdata in IndexedDB; line: MasterdataController.incrementalMasterdataRefresh (http://localhost:3000/static/js/6.chunk.js:10004:10 [^]) (*)
POS2.js:3243 [RefreshMasterdata] The model Color has finished loading a total of 2 records in 16 ms. Loaded in page 1; line: MasterdataController.refreshMasterdata (http://localhost:3000/static/js/6.chunk.js:9853:18 [^]) (*)
POS2.js:3243 Loading page 2 of Color model...; line: MasterdataController.refreshMasterdata (http://localhost:3000/static/js/6.chunk.js:9812:18 [^]) (*)
[incrementalMasterdataRefresh] Finished incremental RefreshMasterdata in IndexedDB; line: MasterdataController.incrementalMasterdataRefresh (http://localhost:3000/static/js/6.chunk.js:10008:10 [^]) (*)

You can see here, that there was a try to load a second page, but it didn't work, no new entries from the second page have been added.
No tags attached.
blocks defect 0047519 closed ranjith_qualiantech_com Incremental Masterdata refresh only loads the first batch of each model, ignoring those in the 2nd+ batch 
Issue History
2021-08-20 12:49marvintmTypedefect => backport
2021-08-20 12:49marvintmTarget Version => RR21Q2.2
2021-08-21 07:20hgbotNote Added: 0131236
2021-08-23 06:11hgbotResolutionopen => fixed
2021-08-23 06:11hgbotStatusscheduled => closed
2021-08-23 06:11hgbotFixed in Version => RR21Q2.2
2021-08-23 06:11hgbotNote Added: 0131243
2021-08-23 06:11hgbotNote Added: 0131244

Notes
(0131236)
hgbot   
2021-08-21 07:20   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/274 [^]
(0131243)
hgbot   
2021-08-23 06:11   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: cdef6a0bd658a1af3675d6f68eeeec70e9b3c159
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 2021-08-21T10:46:43+05:30
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/cdef6a0bd658a1af3675d6f68eeeec70e9b3c159 [^]

Fixed ISSUE-47575: Lastupdated should be updated for each model during incremental refresh
* When incremental refresh, Lastupdated should not updated for each request of a model instead it should be updated for each model

---
M web/org.openbravo.mobile.core/app/model/masterdata/MasterdataController.js
M web/org.openbravo.mobile.core/source/data/ob-datasource.js
---
(0131244)
hgbot   
2021-08-23 06:11   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/274 [^]