Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0051447Retail ModulesWeb POSpublic2023-01-24 15:522023-01-27 07:39
Reportermarvintm 
Assigned Toranjith_qualiantech_com 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR22Q4.2Fixed in VersionRR22Q4.2 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2022-08-29
Regression introduced in releaseRR22Q4
Regression introduced by commithttps://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/9b9213cd7d8c8c10553a4a4b2dfffe1256e970f6 [^]
Triggers an Emergency PackNo
Summary0051447: Cashup update happening in orders and cashups is failing if the "paymentMethods" property is not defined, generating EWIs
DescriptionAs part of this project:

https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/886 [^]

new data was added to the cashup information, and the corresponding classes that process and save it were changed, and they are now expecting it to be there (and specifically, the "paymentMethods" object inside the cashupReportInformation data).

However, if orders or cashups were generated while being offline, and then the customer is updated, this data will not be in the JSON, and the process will fail, generating records in Errors While Importing.

This is not correct, the code should be smart and first check if the "paymentMethods" property exist, and if it doesn't the corresponding code to save it should not be executed.
Steps To Reproduce- Create orders or cashups while being offline in a release before this development
- Update to the newer version with this development, and then synchronize those orders
- Verify that they generate Errors While importing
Proposed SolutionHandle the case of "paymentMethods" property not being present, by just ignoring this part of the code.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0051446 closed ranjith_qualiantech_com Cashup update happening in orders and cashups is failing if the "paymentMethods" property is not defined, generating EWIs 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2023-01-24 15:53marvintmTypedefect => backport
2023-01-24 15:53marvintmTarget Version => RR22Q4.2
2023-01-25 06:05ranjith_qualiantech_comAssigned ToRetail => ranjith_qualiantech_com
2023-01-27 06:34hgbotMerge Request Status => open
2023-01-27 06:34hgbotNote Added: 0145892
2023-01-27 07:39hgbotMerge Request Statusopen => approved
2023-01-27 07:39hgbotResolutionopen => fixed
2023-01-27 07:39hgbotStatusscheduled => closed
2023-01-27 07:39hgbotNote Added: 0145894
2023-01-27 07:39hgbotFixed in Version => RR22Q4.2
2023-01-27 07:39hgbotNote Added: 0145895

Notes
(0145892)
hgbot   
2023-01-27 06:34   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1047 [^]
(0145894)
hgbot   
2023-01-27 07:39   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/1047 [^]
(0145895)
hgbot   
2023-01-27 07:39   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 065a6fdbc1dab2924d8a3cc79d037f451039ebad
Author: Ranjith S R <ranjith@qualiantech.com>
Date: 27-01-2023 10:56:13
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/065a6fdbc1dab2924d8a3cc79d037f451039ebad [^]

Fixed ISSUE-51447: PaymentMethodCashup should updated with paymentMethods if exists
* If paymentMethods is exists in cashup json, then it should be used
  to create the PaymentMethodCashup

---
M src/org/openbravo/retail/posterminal/UpdateCashup.java
---