Openbravo Issue Tracking System - POS2 |
View Issue Details |
|
ID | Project | Category | View Status | Date Submitted | Last Update |
0051282 | POS2 | POS | public | 2023-01-03 15:31 | 2023-03-21 17:08 |
|
Reporter | javierRodriguez | |
Assigned To | meriem_azaf | |
Priority | normal | Severity | major | Reproducibility | always |
Status | closed | Resolution | fixed | |
Platform | | OS | 5 | OS Version | |
Product Version | | |
Target Version | | Fixed in Version | 23Q2 | |
Merge Request Status | approved |
Review Assigned To | |
OBNetwork customer | No |
Support ticket | |
Regression level | |
Regression date | |
Regression introduced in release | |
Regression introduced by commit | |
Triggers an Emergency Pack | No |
|
Summary | 0051282: If localStorage does not have latest safebox information, updated it with the information stored in the backend |
Description | Sometimes, the information about currently attached safebox is not up to date on localStorage (root cause of this will be addressed in another issue). When that happens, we could start the POS:
- assuming that a safebox is attached, when it has already been removed, or
- assuming that there is no safebox attached, when one is currently in the POS
We are going to assume that the backend will always have the most up to date information, so when login in we will:
- make a request to the backend to check if the terminal has currently a safebox attached
- use that information to update the localStorage |
Steps To Reproduce | We can forcefully reproduce the problem like this:
- Configure the terminal to use safeboxes
A) Safebox attached, but localStorage does not have the latest information and thinks it is not attached
- Login, select a safebox
- Logout without doing cashup, safebox will stay in the terminal
- Forcefully remove the safebox by executing: OB.UTIL.localStorage.removeItem('currentSafeBox');
B) Safebox not attached, but localStorage does not have the latest information and thinks it is attached
- Login, select a safebox
- Do a cashup. Check that the safebox has been removed from localStorage by executing OB.UTIL.localStorage.getItem('currentSafeBox'); and checking that it returns null/undefined
- Forcefully update localStorage by executing OB.UTIL.localStorage.removeItem('currentSafeBox');
- Logout without doing cashup, safebox will stay in the terminal
- Forcefully remove the safebox by executing:
OB.UTIL.localStorage.setItem('currentSafeBox','{"safeBoxName":"Vall Blanca Safe Box 03","safeBoxId":"86E4C11FC0CF4A4782E6985AADBD0068","safeBoxCountOnRemove":false,"safeBoxSearchKey":"VBSB03","paymentMethods":[{"allowNotToMove":true,"safeBoxPaymentMethodId":"0A3E31C66F424C09B7BD93624B7439C2","amount":200,"keepFixedAmount":true,"countCash":true,"automateMovementToOtherAccount":true,"allowMoveEverything":true,"countDifferenceLimit":0,"paymentMethodId":"45A202BF44884F05B8A1BF741E2063B6","allowVariableAmount":true,"countPerAmount":false,"currency":"102","isCash":true,"financialAccountId":"16CAAB578E8246E0829DFF987103A3F8","safeBoxCounting":{"depositBalance":0,"paymentBalance":0,"initialBalance":0}}],"id":"86E4C11FC0CF4A4782E6985AADBD0068"}'); |
Proposed Solution | |
Additional Information | |
Tags | No tags attached. |
Relationships | depends on | backport | 0051822 | 23Q1.1 | closed | meriem_azaf | If localStorage does not have latest safebox information, updated it with the information stored in the backend |
|
Attached Files | |
|
Issue History |
Date Modified | Username | Field | Change |
2023-01-03 15:31 | javierRodriguez | New Issue | |
2023-01-03 15:31 | javierRodriguez | Assigned To | => AugustoMauch |
2023-01-03 15:31 | javierRodriguez | OBNetwork customer | => No |
2023-01-03 15:31 | javierRodriguez | Triggers an Emergency Pack | => No |
2023-01-18 14:39 | AugustoMauch | Assigned To | AugustoMauch => meriem_azaf |
2023-01-18 14:46 | AugustoMauch | Summary | Verify terminal values in initial count steps => Add log to inform about the relevant values when deciding what kind of initial count process should be triggered |
2023-01-18 14:46 | AugustoMauch | Description Updated | bug_revision_view_page.php?rev_id=25411#r25411 |
2023-01-18 14:46 | AugustoMauch | Status | new => scheduled |
2023-02-02 12:49 | guilleaer | Assigned To | meriem_azaf => AugustoMauch |
2023-03-07 09:24 | guilleaer | Summary | Add log to inform about the relevant values when deciding what kind of initial count process should be triggered => When state perisitance problem appears -> Mitigate safe box selection |
2023-03-07 09:24 | guilleaer | Description Updated | bug_revision_view_page.php?rev_id=25634#r25634 |
2023-03-09 12:52 | guilleaer | Status | scheduled => acknowledged |
2023-03-09 12:52 | guilleaer | Status | acknowledged => scheduled |
2023-03-13 09:34 | AugustoMauch | Assigned To | AugustoMauch => meriem_azaf |
2023-03-13 09:41 | AugustoMauch | Summary | When state perisitance problem appears -> Mitigate safe box selection => If localStorage does not have latest safebox information, updated it with the information stored in the backend |
2023-03-13 09:41 | AugustoMauch | Description Updated | bug_revision_view_page.php?rev_id=25670#r25670 |
2023-03-13 09:46 | AugustoMauch | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=25672#r25672 |
2023-03-17 15:23 | hgbot | Merge Request Status | => open |
2023-03-17 15:23 | hgbot | Note Added: 0147659 | |
2023-03-17 15:29 | hgbot | Note Added: 0147660 | |
2023-03-20 11:28 | hgbot | Note Added: 0147695 | |
2023-03-21 15:31 | hgbot | Merge Request Status | open => approved |
2023-03-21 15:31 | hgbot | Note Added: 0147765 | |
2023-03-21 15:32 | hgbot | Note Added: 0147766 | |
2023-03-21 15:32 | hgbot | Note Added: 0147767 | |
2023-03-21 15:32 | hgbot | Resolution | open => fixed |
2023-03-21 15:32 | hgbot | Status | scheduled => closed |
2023-03-21 15:32 | hgbot | Note Added: 0147768 | |
2023-03-21 15:32 | hgbot | Fixed in Version | => 23Q2 |
2023-03-21 15:32 | hgbot | Note Added: 0147769 | |
2023-03-21 17:08 | hgbot | Note Added: 0147776 | |
Notes |
|
(0147659)
|
hgbot
|
2023-03-17 15:23
|
|
|
|
(0147660)
|
hgbot
|
2023-03-17 15:29
|
|
|
|
(0147695)
|
hgbot
|
2023-03-20 11:28
|
|
|
|
(0147765)
|
hgbot
|
2023-03-21 15:31
|
|
|
|
(0147766)
|
hgbot
|
2023-03-21 15:32
|
|
|
|
(0147767)
|
hgbot
|
2023-03-21 15:32
|
|
|
|
(0147768)
|
hgbot
|
2023-03-21 15:32
|
|
|
|
(0147769)
|
hgbot
|
2023-03-21 15:32
|
|
Directly closing issue as related merge request is already approved.
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 2503c032165bba335e1ec98a343da1bf406cd6ae
Author: Meriem Azaf <meriem.azaf@openbravo.com>
Date: 21-03-2023 14:44:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/2503c032165bba335e1ec98a343da1bf406cd6ae [^]
Fixes ISSUE-51282: Not matching safeBox information between the localStorage and the backend
Sometimes, the information about currently attached safebox is not up to date on localStorage. When this has happened, the mechanism to mitigate the same
problem but applied to the cashup information is triggered.
In this issue, if the cashup mitigation was triggered, a request will be done to the backend to find out if the backend thinks there is a safebox attached
to the terminal. If there is not, but there seems to be a safebox attached on the local data, we will assume the local data is outdated and we will remove
the information about the currently attached safebox stored locally.
---
A src/org/openbravo/pos2/master/SafeBox.java
M src-db/database/sourcedata/OBMOBC_SERVICES.xml
M web-jspack/org.openbravo.pos2/src/authentication/loginInitActions.js
---
|
|
|
(0147776)
|
hgbot
|
2023-03-21 17:08
|
|
|