Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0048411 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [Retail Modules] Web POS | major | always | 2022-01-13 18:39 | 2022-01-20 16:44 | |||
| Reporter | gorka_gil | View Status | public | |||||
| Assigned To | ranjith_qualiantech_com | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | RR21Q4.4 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | RR21Q4.4 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0048411: 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 | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0134409) hgbot (developer) 2022-01-20 12:22 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/695 [^] |
|
(0134424) hgbot (developer) 2022-01-20 16:44 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^] Changeset: 86805e122a93a298603b10b703f44f02db1637b1 Author: Ranjith S R <ranjith@qualiantech.com> Date: 20-01-2022 16:47:33 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/86805e122a93a298603b10b703f44f02db1637b1 [^] Fixed ISSUE-48411: 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 --- |
|
(0134425) hgbot (developer) 2022-01-20 16:44 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/695 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2022-01-17 16:50 | marvintm | Type | defect => backport |
| 2022-01-17 16:50 | marvintm | Target Version | => RR21Q4.4 |
| 2022-01-20 12:22 | hgbot | Note Added: 0134409 | |
| 2022-01-20 16:44 | hgbot | Resolution | open => fixed |
| 2022-01-20 16:44 | hgbot | Status | scheduled => closed |
| 2022-01-20 16:44 | hgbot | Fixed in Version | => RR21Q4.4 |
| 2022-01-20 16:44 | hgbot | Note Added: 0134424 | |
| 2022-01-20 16:44 | hgbot | Note Added: 0134425 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |