Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0052833 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| backport | [POS2] POS | minor | have not tried | 2023-06-22 14:19 | 2023-06-23 09:51 | |||
| Reporter | guilleaer | View Status | public | |||||
| Assigned To | guilleaer | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 23Q1.4 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | |||||
| Projection | none | ETA | none | Target Version | 23Q1.4 | |||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0052833: Using a payment method defined as a safebox the check to ensure that enough cash is available for a return should be skipped | |||||||
| Description | Using a payment method defined as a safebox the check to ensure that enough cash is available for a return should be skipped | |||||||
| Steps To Reproduce | In 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 Solution | For safeboxes we can't know the content of the safebox so this check should not be done | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0151516) hgbot (developer) 2023-06-22 18:54 |
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1915 [^] |
|
(0151545) 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: 593866cb17a0f5058b6922aba7c760b53bb2b483 Author: Guillermo Alvarez de Eulate <guillermo.alvarez@openbravo.com> Date: 23-06-2023 07:51:19 URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/commit/593866cb17a0f5058b6922aba7c760b53bb2b483 [^] fixed ISSUE-52833: Payment methods defined as safe boxes should not check cash... --- M web-jspack/org.openbravo.pos2/src/components/Payments/PaymentIntegration/processPayment.js --- |
|
(0151546) hgbot (developer) 2023-06-23 09:51 |
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.pos2/-/merge_requests/1915 [^] |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2023-06-22 14:19 | guilleaer | Type | defect => backport |
| 2023-06-22 14:19 | guilleaer | Target Version | => 23Q1.4 |
| 2023-06-22 18:54 | hgbot | Note Added: 0151516 | |
| 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 | => 23Q1.4 |
| 2023-06-23 09:51 | hgbot | Note Added: 0151545 | |
| 2023-06-23 09:51 | hgbot | Note Added: 0151546 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |