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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0052831
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[POS2] POSminorhave not tried2023-06-22 14:192023-06-23 09:51
ReporterguilleaerView Statuspublic 
Assigned Toguilleaer 
PrioritynormalResolutionfixedFixed in Version23Q3
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0052831: 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 ]
depends on backport 005283223Q2.2 closedguilleaer Using a payment method defined as a safebox the check to ensure that enough cash is available for a return should be skipped 
depends on backport 005283323Q1.4 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
(0151506)
hgbot (developer)
2023-06-22 14:38

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

Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1913 [^]
(0151548)
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: 6602f9f24bb22b0cd39a4a2d1b67da572fa7e699
Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com>
Date: 23-06-2023 07:51:29
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/6602f9f24bb22b0cd39a4a2d1b67da572fa7e699 [^]

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

---
M web-jspack/org.openbravo.pos2/src/components/Payments/PaymentIntegration/processPayment.js
---

- Issue History
Date Modified Username Field Change
2023-06-22 14:19 guilleaer New Issue
2023-06-22 14:19 guilleaer Assigned To => guilleaer
2023-06-22 14:19 guilleaer Triggers an Emergency Pack => No
2023-06-22 14:19 guilleaer Status new => scheduled
2023-06-22 14:38 hgbot Note Added: 0151506
2023-06-23 09:51 hgbot Resolution open => fixed
2023-06-23 09:51 hgbot Status scheduled => closed
2023-06-23 09:51 hgbot Note Added: 0151547
2023-06-23 09:51 hgbot Fixed in Version => 23Q3
2023-06-23 09:51 hgbot Note Added: 0151548


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker