Openbravo Issue Tracking System - Retail Modules
View Issue Details
0048410Retail ModulesWeb POSpublic2022-01-13 18:392022-01-20 16:43
gorka_gil 
ranjith_qualiantech_com 
normalmajoralways
closedfixed 
5
 
RR22Q1RR22Q1 
No
0048410: Cannot send back to the backend a paid order opened from a previous cashup
When opening an order from a previous cashup and sending back to the backend it gives this error:

"The cashup related to this order has been processed"
Create a ticket
Do cashup
Open the previous ticket
Click on pay and done

It will produce an error while importing:
"The cashup related to this order has been processed"
When sending the json to the backend, in the json are included two cashup id's:
   - obposAppCashup : it is the cashup id linked to the order in the tables
   - cashUpReportInformation[id] : it is the cashup information of the pos when sending the ticket to the backend

If loading a order from a previous cashup, the cashUpReportInformation[id] will point to the current cashup, but the obposAppCashup will point to the cashup when the orded was created

So the proposal is change this check to instead of check the obposAppCashup , to check the cashUpReportInformation[id]

Attached fix proposal
No tags attached.
blocks defect 0048399 closed ranjith_qualiantech_com Cannot send back to the backend a paid order opened from a previous cashup 
Issue History
2022-01-17 16:50marvintmTypedefect => backport
2022-01-17 16:50marvintmTarget Version => RR22Q1
2022-01-20 12:22hgbotNote Added: 0134408
2022-01-20 16:43hgbotResolutionopen => fixed
2022-01-20 16:43hgbotStatusscheduled => closed
2022-01-20 16:43hgbotFixed in Version => RR22Q1
2022-01-20 16:43hgbotNote Added: 0134422
2022-01-20 16:43hgbotNote Added: 0134423

Notes
(0134408)
hgbot   
2022-01-20 12:22   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/694 [^]
(0134422)
hgbot   
2022-01-20 16:43   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 867b55fa747277284e235100e041657016e17f9f
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 20-01-2022 16:47:27
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/867b55fa747277284e235100e041657016e17f9f [^]

Fixed ISSUE-48410: Cashup should be validated when processing loaded order
* When processing loaded order, Cashup should be validated with cashUpReportInformation
  instead of obposAppCashup from json

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---
(0134423)
hgbot   
2022-01-20 16:43   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/694 [^]