Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0048399
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2022-01-13 18:392022-01-19 13:14
Reportergorka_gilView Statuspublic 
Assigned Toranjith_qualiantech_com 
PrioritynormalResolutionfixedFixed in VersionRR22Q2
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression levelProduction - Confirmed Stable
Regression date2021-02-09
Regression introduced in releaseRR21Q2
Regression introduced by commithttps://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/39054d0461fd122a5e8d2e386d1d1cafcd82421e [^]
Triggers an Emergency PackNo
Summary

0048399: Cannot send back to the backend a paid order opened from a previous cashup

DescriptionWhen 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 ReproduceCreate 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 SolutionWhen 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
TagsNo tags attached.
Attached Filesdiff file icon fixLoadPaidOrder-posterminal.diff [^] (1,948 bytes) 2022-01-13 18:41 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
depends on backport 0048410RR22Q1 closedranjith_qualiantech_com Cannot send back to the backend a paid order opened from a previous cashup 
depends on backport 0048411RR21Q4.4 closedranjith_qualiantech_com Cannot send back to the backend a paid order opened from a previous cashup 
caused by defect 0045579 closedranjith_qualiantech_com Wrong cashup_id link for a ticket when doing an action 

-  Notes
(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 [^]

- Issue History
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 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: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 Note Added: 0134372
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
Powered by Mantis Bugtracker