Openbravo Issue Tracking System - Retail Modules
View Issue Details
0050362Retail ModulesWeb POSpublic2022-09-27 23:492022-09-28 23:37
AugustoMauch 
AugustoMauch 
normalmajorrandom
closedfixed 
5
 
RR22Q3.1RR22Q3.1 
No
0050362: Add a check to automatically restart a cashup if current one is already processed
We have detected that sometimes there is a problem with the state persistence and as a result after completing a cashup, the cashup that was just completed remains in the status of the application. Then, the next time the cashier logs in he will keep using that cashup, and he will end up with an error in the backend after trying to process a cashup that has already been processed.

We are still trying to find the root cause (state not being persisted properly), but in this issue we mean to prevent the duplicated cashup processing by checking if the user is working with a processed cashup and creating and in that case creating a new one
We are not able to reproduce the issue unless we:
- Increase the throttle of the state persistence from 100ms to 10s
- Remove the state persistence flush we do after each cashup completion

Doing that, if the user closes or refreshes the application right after completing the cashup, the completion will not be persisted and the next time the user logs in he will keep using the previous version of the cashup
No tags attached.
blocks defect 0050361 closed AugustoMauch Add a check to automatically restart a cashup if current one is already processed 
Issue History
2022-09-27 23:49AugustoMauchTypedefect => backport
2022-09-27 23:49AugustoMauchTarget Version => RR22Q3.1
2022-09-28 12:47priyamIssue Monitored: priyam
2022-09-28 23:30hgbotNote Added: 0141463
2022-09-28 23:32hgbotNote Added: 0141464
2022-09-28 23:37hgbotNote Added: 0141465
2022-09-28 23:37hgbotNote Added: 0141466
2022-09-28 23:37hgbotNote Added: 0141467
2022-09-28 23:37hgbotResolutionopen => fixed
2022-09-28 23:37hgbotStatusscheduled => closed
2022-09-28 23:37hgbotFixed in Version => RR22Q3.1
2022-09-28 23:37hgbotNote Added: 0141468

Notes
(0141463)
hgbot   
2022-09-28 23:30   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/413 [^]
(0141464)
hgbot   
2022-09-28 23:32   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/942 [^]
(0141465)
hgbot   
2022-09-28 23:37   
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core [^]
Changeset: 36c153d52adc14713d7d2c1c229b0105da1122e3
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 28-09-2022 23:30:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/commit/36c153d52adc14713d7d2c1c229b0105da1122e3 [^]

Related to ISSUE-50362: Extracts TerminalLog.addLog to utility function

---
M web/org.openbravo.mobile.core/app/model/business-object/terminal-log/TerminalLogUtils.js
M web/org.openbravo.mobile.core/app/model/business-object/terminal-log/actions/AddLog.js
---
(0141466)
hgbot   
2022-09-28 23:37   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/413 [^]
(0141467)
hgbot   
2022-09-28 23:37   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/942 [^]
(0141468)
hgbot   
2022-09-28 23:37   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 9069e5d237ae8c40c115f7dc3c032479a5e36d02
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 28-09-2022 23:32:04
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/9069e5d237ae8c40c115f7dc3c032479a5e36d02 [^]

Fixes ISSUE-50362: Prevents duplicated cashup

In order to prevent a duplicated cashup if there has been a problem
persisting the last cashup completion, we will now detect the case when
the current cashup has already been processed in the backend, and in
that case we will start a new one from scratch in the backend.

---
M web/org.openbravo.retail.posterminal/app/model/business-object/cashup/actions/InitCashup.js
---