Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0050305Retail ModulesWeb POSpublic2022-09-21 17:562022-09-29 12:44
Reporteraaroncalero 
Assigned Toigor_trebol 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR22Q4 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0050305: Payment Methods for Refunds: originalReceiptPayments replaced after adding several verified return lines
DescriptionUsing the Payment Method for Refunds functionality, the originalReceiptPayments object is used to track the payment method(s) used in the original receipt when doing a verified return.

When adding verified return lines from different original tickets, the originalReceiptPayments property is replaced by information of the most recent line.
Steps To ReproduceLog in backoffice, go to the Channel - Touchpoint Type window and configure the Payment Method for Refunds functionality:
* Add Cash as the PM for Refund for Cash
* Add Card as the PM for Refund for Card

Log in Webpos and create two different tickets:
One with a single product and paid totally in Cash
A second ticket with a single product and paid totally in Card.

In a new ticket, go to the menu > Verified Returns and return the first ticket.
Verify that the originalReceiptPayments property contains the Cash payment method.
Go again to menu > Verified returns and return the second ticket.
Verify again the originalReceiptPayments property and notice that it now contains only Card.
Proposed SolutionTo keep the functionality consistent with what happens to originalReceiptPayments when paying the original order with multiple payment methods, when adding several verified return lines the originalReceiptPayments object should merge all payment methods.
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2022-09-21 17:56aaroncaleroNew Issue
2022-09-21 17:56aaroncaleroAssigned To => Retail
2022-09-21 17:56aaroncaleroOBNetwork customer => No
2022-09-21 17:56aaroncaleroTriggers an Emergency Pack => No
2022-09-22 12:19aaroncaleroAssigned ToRetail => alainperez
2022-09-23 14:57hgbotMerge Request Status => open
2022-09-23 14:57hgbotNote Added: 0141323
2022-09-23 15:07hgbotNote Added: 0141324
2022-09-23 15:09hgbotNote Added: 0141325
2022-09-23 15:10hgbotNote Added: 0141326
2022-09-26 18:24aferrazAssigned Toalainperez => igor_trebol
2022-09-29 12:43hgbotMerge Request Statusopen => approved
2022-09-29 12:44hgbotNote Added: 0141502
2022-09-29 12:44hgbotResolutionopen => fixed
2022-09-29 12:44hgbotStatusnew => closed
2022-09-29 12:44hgbotFixed in Version => RR22Q4
2022-09-29 12:44hgbotNote Added: 0141503
2022-09-29 12:44hgbotNote Added: 0141504
2022-09-29 12:44hgbotNote Added: 0141505

Notes
(0141323)
hgbot   
2022-09-23 14:57   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/929 [^]
(0141324)
hgbot   
2022-09-23 15:07   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/42 [^]
(0141325)
hgbot   
2022-09-23 15:09   
Merge request closed: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/42 [^]
(0141326)
hgbot   
2022-09-23 15:10   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/43 [^]
(0141502)
hgbot   
2022-09-29 12:44   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/929 [^]
(0141503)
hgbot   
2022-09-29 12:44   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 84bc73a0b0e28b4f697fe2c4a2f28f822962b2e2
Author: Igor Trebol <igor.trebol@openbravo.com>
Date: 29-09-2022 10:44:16
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/84bc73a0b0e28b4f697fe2c4a2f28f822962b2e2 [^]

Fixes ISSUE-50305: Solved problem with incorrect original ticket data in Verified returns

---
M web-test/model/business-object/ticket/TicketUtils.test.js
M web/org.openbravo.retail.posterminal/app/model/business-object/ticket/AddPaymentUtils.js
M web/org.openbravo.retail.posterminal/app/model/business-object/ticket/TicketUtils.js
---
(0141504)
hgbot   
2022-09-29 12:44   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns [^]
Changeset: 3f8765d69cdfdee373833caa0ce8f27be9862af1
Author: Igor Trebol <igor.trebol@openbravo.com>
Date: 23-09-2022 15:11:02
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/commit/3f8765d69cdfdee373833caa0ce8f27be9862af1 [^]

Fixes ISSUE-50305: Solved problem with incorrect original ticket data in Verified returns

---
M web/org.openbravo.retail.returns/app/model/business-object/ticket/actions/ReturnVerifiedTicket.js
---
(0141505)
hgbot   
2022-09-29 12:44   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.returns/-/merge_requests/43 [^]