Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0048635 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Cash up | major | always | 2022-02-18 15:28 | 2022-04-25 12:05 | |||
| Reporter | ibrahim_hammani | View Status | public | |||||
| Assigned To | rqueralta | |||||||
| Priority | high | Resolution | fixed | Fixed in Version | RR22Q2 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | approved | |||||||
| Review Assigned To | marvintm | |||||||
| OBNetwork customer | Gold | |||||||
| Support ticket | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0048635: Wrong values in Backoffice cashup report | |||||||
| Description | In the Backoffice cashup report we have some wrong values on the cash payment method that leads to wrong values on totals. Ther wrong values are in: 1- Starting amount (cash part and total) 2- Withdrawal amount (cash part and total) 3- Expected amount (cash part and total) It will be wise to check out the other values too and do further testing. | |||||||
| Steps To Reproduce | 1. Log in to Web POS 2. You see the open till screen 3. Add cash until the counted amount becomes equals to the expected amount 4. Hit Don button 5. Perform a cash-up with a surplus of X amount 6. Add a product to the current ticket and ensure it contains some fraction digits (cents) 7. Click checkout and pay the total amount in cash 8. Go to menu and clik Cashup 9. Add cash until : counted amount =expect amount - cents; and remaining is only cents example: expected 290.95 and counted 290.00 ant remainning is 0.95 10. hit next and continue cashup until the end. 11. Log in to the back office and go to the Touchpoint window and look for the same terminal used for creating a ticket 12. Navigate to the cash up history tab and look for the current day's record 13. Click on the Cashup Report button and generate the PDF report 14. Notice that the Starting cash amount, Withdrawal Cash amount and the Expected cash amount does't display the right values | |||||||
| Proposed Solution | All the math for that report is done in this method org.openbravo.retail.sessions.CashupRepHook.CashupReportHookResult exec(OBPOSAppCashup cashup,List<HashMap<String, String>> hashMapList, HashMap<String, Object> parameters) | |||||||
| Tags | No tags attached. | |||||||
| Attached Files |  Selection_028.png [^] (95,616 bytes) 2022-03-10 12:19
Selection_027.png [^] (34,245 bytes) 2022-03-10 12:19
| |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0135237) hgbot (developer) 2022-02-22 16:38 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/merge_requests/36 [^] |
|
(0135294) hgbot (developer) 2022-02-24 17:42 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions [^] Changeset: 9b247a93a2e22d9db597ed25a4d6b7307a034878 Author: Rafael Queralta <rafaelcuba81@gmail.com> Date: 21-02-2022 23:14:13 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/commit/9b247a93a2e22d9db597ed25a4d6b7307a034878 [^] Fixed BUG-48635: Do not subtract the "drop" value to the "Starting" and "Expected" value --- M src/org/openbravo/retail/sessions/CashupRepHook.java --- |
|
(0135295) hgbot (developer) 2022-02-24 17:42 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/merge_requests/36 [^] |
|
(0135617) yogaskarnik (viewer) 2022-03-10 12:18 |
The report shows duplicate withdrawal entries. In the back office, I can see a double entry with a milliseconds difference. Attaching are the screenshots |
|
(0135666) hgbot (developer) 2022-03-14 03:50 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/merge_requests/37 [^] |
|
(0135820) hgbot (developer) 2022-03-17 19:19 |
Merge request closed: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/merge_requests/37 [^] |
|
(0135821) hgbot (developer) 2022-03-17 19:24 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/merge_requests/38 [^] |
|
(0135828) hgbot (developer) 2022-03-18 07:48 |
Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions [^] Changeset: 3fd2d4d1218ccdd4751a8d83b6ae947600afeb00 Author: Rafael Queralta <rafaelcuba81@gmail.com> Date: 17-03-2022 14:22:08 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/commit/3fd2d4d1218ccdd4751a8d83b6ae947600afeb00 [^] Related to BUG-48635: Check if drop or deposit session was added before run cashup hook --- M src/org/openbravo/retail/sessions/CashupRepHook.java --- |
|
(0135829) hgbot (developer) 2022-03-18 07:48 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.sessions/-/merge_requests/38 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2022-02-18 15:28 | ibrahim_hammani | New Issue | |
| 2022-02-18 15:28 | ibrahim_hammani | Assigned To | => rqueralta |
| 2022-02-18 15:28 | ibrahim_hammani | OBNetwork customer | => OBPS |
| 2022-02-18 15:28 | ibrahim_hammani | Triggers an Emergency Pack | => No |
| 2022-02-21 07:12 | guillermogil | OBNetwork customer | OBPS => Gold |
| 2022-02-22 16:28 | rqueralta | Status | new => scheduled |
| 2022-02-22 16:38 | hgbot | Merge Request Status | => open |
| 2022-02-22 16:38 | hgbot | Note Added: 0135237 | |
| 2022-02-24 17:42 | hgbot | Merge Request Status | open => approved |
| 2022-02-24 17:42 | hgbot | Resolution | open => fixed |
| 2022-02-24 17:42 | hgbot | Status | scheduled => closed |
| 2022-02-24 17:42 | hgbot | Note Added: 0135294 | |
| 2022-02-24 17:42 | hgbot | Note Added: 0135295 | |
| 2022-03-10 12:18 | yogaskarnik | Note Added: 0135617 | |
| 2022-03-10 12:18 | yogaskarnik | Status | closed => new |
| 2022-03-10 12:18 | yogaskarnik | Resolution | fixed => open |
| 2022-03-10 12:19 | yogaskarnik | File Added: Selection_028.png | |
| 2022-03-10 12:19 | yogaskarnik | File Added: Selection_027.png | |
| 2022-03-14 03:50 | hgbot | Note Added: 0135666 | |
| 2022-03-17 19:19 | hgbot | Note Added: 0135820 | |
| 2022-03-17 19:24 | hgbot | Note Added: 0135821 | |
| 2022-03-18 07:48 | hgbot | Note Added: 0135828 | |
| 2022-03-18 07:48 | hgbot | Note Added: 0135829 | |
| 2022-03-18 07:48 | marvintm | Status | new => scheduled |
| 2022-03-18 07:49 | marvintm | Status | scheduled => resolved |
| 2022-03-18 07:49 | marvintm | Fixed in Version | => RR22Q2 |
| 2022-03-18 07:49 | marvintm | Resolution | open => fixed |
| 2022-03-18 07:49 | marvintm | Review Assigned To | => marvintm |
| 2022-03-18 07:49 | marvintm | Status | resolved => closed |
| 2022-04-12 09:26 | yogaskarnik | Resolution | fixed => open |
| 2022-04-25 12:05 | marvintm | Resolution | open => fixed |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |