Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0048399 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | major | always | 2022-01-13 18:39 | 2022-01-19 13:14 | |||
Reporter | gorka_gil | View Status | public | |||||
Assigned To | ranjith_qualiantech_com | |||||||
Priority | normal | 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 | ||||||||
OBNetwork customer | Gold | |||||||
Support ticket | ||||||||
Regression level | Production - Confirmed Stable | |||||||
Regression date | 2021-02-09 | |||||||
Regression introduced in release | RR21Q2 | |||||||
Regression introduced by commit | https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/39054d0461fd122a5e8d2e386d1d1cafcd82421e [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0048399: Cannot send back to the backend a paid order opened from a previous cashup | |||||||
Description | 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" | |||||||
Steps To Reproduce | 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" | |||||||
Proposed Solution | 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 | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
||||||||||||||||||||||
|
![]() |
|
(0134372) hgbot (developer) 2022-01-19 06:19 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/693 [^] |
(0134385) hgbot (developer) 2022-01-19 13:14 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^] Changeset: 570fd81f2aea280c1a638c48a14d80874d138b74 Author: Ranjith S R <ranjith@qualiantech.com> Date: 18-01-2022 12:51:36 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/570fd81f2aea280c1a638c48a14d80874d138b74 [^] Fixed ISSUE-48399: 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 --- |
(0134386) hgbot (developer) 2022-01-19 13:14 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/693 [^] |
![]() |
|||
Date Modified | Username | Field | Change |
2022-01-13 18:39 | gorka_gil | New Issue | |
2022-01-13 18:39 | gorka_gil | Assigned To | => Retail |
2022-01-13 18:39 | gorka_gil | OBNetwork customer | => No |
2022-01-13 18:39 | gorka_gil | Triggers an Emergency Pack | => No |
2022-01-13 18:40 | gorka_gil | Status | new => acknowledged |
2022-01-13 18:41 | gorka_gil | File Added: fixLoadPaidOrder-posterminal.diff | |
2022-01-13 18:41 | gorka_gil | Relationship added | caused by 0045579 |
2022-01-17 08:33 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
2022-01-17 16:48 | marvintm | OBNetwork customer | No => Gold |
2022-01-17 16:50 | marvintm | Status | acknowledged => scheduled |
2022-01-17 16:50 | marvintm | Regression level | => Production - Confirmed Stable |
2022-01-17 16:50 | marvintm | Regression date | => 2021-02-09 |
2022-01-17 16:50 | marvintm | Regression introduced in release | => RR21Q2 |
2022-01-17 16:50 | marvintm | Regression introduced by commit | => https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/39054d0461fd122a5e8d2e386d1d1cafcd82421e [^] |
2022-01-19 06:19 | hgbot | Merge Request Status | => open |
2022-01-19 06:19 | hgbot | Note Added: 0134372 | |
2022-01-19 13:14 | hgbot | Merge Request Status | open => approved |
2022-01-19 13:14 | hgbot | Resolution | open => fixed |
2022-01-19 13:14 | hgbot | Status | scheduled => closed |
2022-01-19 13:14 | hgbot | Fixed in Version | => RR22Q2 |
2022-01-19 13:14 | hgbot | Note Added: 0134385 | |
2022-01-19 13:14 | hgbot | Note Added: 0134386 |
Copyright © 2000 - 2009 MantisBT Group |