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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0052832
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[POS2] POSminorhave not tried2023-06-22 14:192023-06-23 09:51
ReporterguilleaerView Statuspublic 
Assigned Toguilleaer 
PrioritynormalResolutionfixedFixed in Version23Q2.2
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version23Q2.2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerGold
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0052832: Using a payment method defined as a safebox the check to ensure that enough cash is available for a return should be skipped

DescriptionUsing a payment method defined as a safebox the check to ensure that enough cash is available for a return should be skipped
Steps To ReproduceIn POS2 environemnt using VBS-2 terminal
Configure system to use safeboxes
(touchpoint type -> Use safe boxes)
(Touchpoint type - payment method (cash)) -> defined as safebox

Do login in POS
Select a safebox in the cashup
Review the starting cash of the payment method to make sure it is 0 for cash
(OB.App.State.getState().Cashup.cashPaymentMethodInfo)
Do a sale of Insect Repellent 14.50 paying by cash
Review the payment method to make sure it is 0 for cash and collections are 14.50
(OB.App.State.getState().Cashup.cashPaymentMethodInfo)
Try to do a return of the insect repellent. When payment is done by cash no issues -> OK
Remove the payment and go back
Change the price of the product to 15
Go to pay and add the negative payment by cash
Error is shown because there is no enough cash -> Error

For safeboxes we can't know the content of the safebox so this check should not be done

Proposed SolutionFor safeboxes we can't know the content of the safebox so this check should not be done
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
blocks defect 0052831 closedguilleaer Using a payment method defined as a safebox the check to ensure that enough cash is available for a return should be skipped 

-  Notes
(0151515)
hgbot (developer)
2023-06-22 18:54

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1914 [^]
(0151543)
hgbot (developer)
2023-06-23 09:51

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2 [^]
Changeset: 62a967254bc622239ce85d955ba20efbf9dedc9a
Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
Date: 23-06-2023 07:51:15
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/62a967254bc622239ce85d955ba20efbf9dedc9a [^]

fixed ISSUE-52832: Payment methods defined as safe boxes should not check cash...

---
M web-jspack/org.openbravo.pos2/src/components/Payments/PaymentIntegration/processPayment.js
---
(0151544)
hgbot (developer)
2023-06-23 09:51

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1914 [^]

- Issue History
Date Modified Username Field Change
2023-06-22 14:19 guilleaer Type defect => backport
2023-06-22 14:19 guilleaer Target Version => 23Q2.2
2023-06-22 18:54 hgbot Merge Request Status => open
2023-06-22 18:54 hgbot Note Added: 0151515
2023-06-23 08:56 hgbot Merge Request Status open => approved
2023-06-23 09:51 hgbot Resolution open => fixed
2023-06-23 09:51 hgbot Status scheduled => closed
2023-06-23 09:51 hgbot Fixed in Version => 23Q2.2
2023-06-23 09:51 hgbot Note Added: 0151543
2023-06-23 09:51 hgbot Note Added: 0151544

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker