Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0050362
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] Web POSmajorrandom2022-09-27 23:492022-09-28 23:37
ReporterAugustoMauchView Statuspublic 
Assigned ToAugustoMauch 
PrioritynormalResolutionfixedFixed in VersionRR22Q3.1
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget VersionRR22Q3.1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0050362: Add a check to automatically restart a cashup if current one is already processed

DescriptionWe 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
Steps To ReproduceWe 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
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0050361 closedAugustoMauch Add a check to automatically restart a cashup if current one is already processed 

-  Notes
(0141463)
hgbot (developer)
2022-09-28 23:30

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/413 [^]
(0141464)
hgbot (developer)
2022-09-28 23:32

Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/942 [^]
(0141465)
hgbot (developer)
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 (developer)
2022-09-28 23:37

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.mobile.core/-/merge_requests/413 [^]
(0141467)
hgbot (developer)
2022-09-28 23:37

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/942 [^]
(0141468)
hgbot (developer)
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
---

- Issue History
Date Modified Username Field Change
2022-09-27 23:49 AugustoMauch Type defect => backport
2022-09-27 23:49 AugustoMauch Target Version => RR22Q3.1
2022-09-28 12:47 priyam Issue Monitored: priyam
2022-09-28 23:30 hgbot Note Added: 0141463
2022-09-28 23:32 hgbot Note Added: 0141464
2022-09-28 23:37 hgbot Note Added: 0141465
2022-09-28 23:37 hgbot Note Added: 0141466
2022-09-28 23:37 hgbot Note Added: 0141467
2022-09-28 23:37 hgbot Resolution open => fixed
2022-09-28 23:37 hgbot Status scheduled => closed
2022-09-28 23:37 hgbot Fixed in Version => RR22Q3.1
2022-09-28 23:37 hgbot Note Added: 0141468


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker