Openbravo Issue Tracking System - Retail Modules | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0036977 | Retail Modules | Web POS | public | 2017-09-29 14:36 | 2018-01-22 20:57 |
| Reporter | jorgewederago | ||||
| Assigned To | rqueralta | ||||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | |||||
| Target Version | Fixed in Version | RR18Q1 | |||
| Merge Request Status | |||||
| Review Assigned To | guilleaer | ||||
| OBNetwork customer | |||||
| Support ticket | |||||
| Regression level | |||||
| Regression date | |||||
| Regression introduced in release | |||||
| Regression introduced by commit | |||||
| Triggers an Emergency Pack | No | ||||
| Summary | 0036977: Under certain circumstances an overpayment is not evaluated properly | ||||
| Description | If you set a maximum amount for overpayment/max limit change it can be ignored under certain circumstances. | ||||
| Steps To Reproduce | Backend: *Go to backend > POSTerminal Type window. *Open Credit Card payment method. *Set Overpayment/MaxLimit Change as 100 WebPOS: *Add Avalanche Transceiver *Add credit card payment with 500 as value *Check that an error message appears and Done button is disabled *Remove the payment and add a new one with Credit Card with value 5000 In this case you will be able to close the transaction ignoring the maxLimit Change/Overpayment value. | ||||
| Proposed Solution | } else if (selectedPayment.paymentMethod.overpaymentLimit < requiredCash) { The line above compares a number against an string, this wont work if the string contains both "," and "." symbols. It might be a good idea to convert requiredCash to a number too. | ||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2017-09-29 14:36 | jorgewederago | New Issue | |||
| 2017-09-29 14:36 | jorgewederago | Assigned To | => Retail | ||
| 2017-09-29 14:36 | jorgewederago | Triggers an Emergency Pack | => No | ||
| 2017-09-29 14:49 | marvintm | Resolution time | => 1508450400 | ||
| 2017-10-10 15:52 | rqueralta | Assigned To | Retail => rqueralta | ||
| 2017-10-10 15:53 | rqueralta | Status | new => scheduled | ||
| 2017-10-18 16:24 | hgbot | Checkin | |||
| 2017-10-18 16:24 | hgbot | Note Added: 0099941 | |||
| 2017-10-18 16:24 | hgbot | Status | scheduled => resolved | ||
| 2017-10-18 16:24 | hgbot | Resolution | open => fixed | ||
| 2017-10-18 16:24 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/ae5f211d6c8fee8aa7d834f5a41ee6be95dfa591 [^] | ||
| 2017-10-18 16:46 | hgbot | Checkin | |||
| 2017-10-18 16:46 | hgbot | Note Added: 0099943 | |||
| 2017-10-23 15:00 | guilleaer | Review Assigned To | => guilleaer | ||
| 2017-10-23 15:00 | guilleaer | Status | resolved => closed | ||
| 2017-10-23 15:00 | guilleaer | Fixed in Version | => RR18Q1 | ||
| 2017-11-30 15:25 | hgbot | Checkin | |||
| 2017-11-30 15:25 | hgbot | Note Added: 0100839 | |||
| 2018-01-22 20:57 | hgbot | Checkin | |||
| 2018-01-22 20:57 | hgbot | Note Added: 0101866 | |||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||