Openbravo Issue Tracking System - Retail Modules | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0029221 | Retail Modules | Return Receipt | public | 2015-03-10 13:28 | 2015-03-30 12:38 |
| Reporter | migueldejuana | ||||
| Assigned To | migueldejuana | ||||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | OS | 5 | OS Version | ||
| Product Version | |||||
| Target Version | Fixed in Version | RR15Q2 | |||
| Merge Request Status | |||||
| Review Assigned To | marvintm | ||||
| OBNetwork customer | |||||
| Support ticket | |||||
| Regression level | |||||
| Regression date | |||||
| Regression introduced in release | |||||
| Regression introduced by commit | |||||
| Triggers an Emergency Pack | No | ||||
| Summary | 0029221: Discounts wrong calculated when splitting lines in returns | ||||
| Description | We are no calculating well discounts when returning a verified return. We are using remaining quantity instead of total line qty. | ||||
| Steps To Reproduce | - Activate "Split Lines in Shipments when Returning" preference - Login - Create an order with stock in 2 different bins adding 10% discount - Press Verified Returns in the menu - Select previous order and return part of the qty - Complete the return - Press Verified Returns in the menu - Select previous order and return all qty - ERROR: See discounts applied | ||||
| Proposed Solution | Replace: splitpromo.amt = OB.DEC.mul(p.amt, (splitline.remainingQuantity / line.remainingQuantity)); splitpromo.actualAmt = OB.DEC.mul(p.actualAmt, (splitline.remainingQuantity / line.remainingQuantity)); With: splitpromo.amt = OB.DEC.mul(p.amt, (splitline.remainingQuantity / line.quantity)); splitpromo.actualAmt = OB.DEC.mul(p.actualAmt, (splitline.remainingQuantity / line.quantity)); | ||||
| Additional Information | |||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 2015-03-10 13:28 | migueldejuana | New Issue | |||
| 2015-03-10 13:28 | migueldejuana | Assigned To | => malsasua | ||
| 2015-03-10 13:28 | migueldejuana | Regression level | => Production - QA Approved | ||
| 2015-03-10 13:28 | migueldejuana | Triggers an Emergency Pack | => No | ||
| 2015-03-12 16:23 | migueldejuana | Regression level | Production - QA Approved => | ||
| 2015-03-17 09:21 | migueldejuana | Assigned To | malsasua => migueldejuana | ||
| 2015-03-17 09:56 | hgbot | Checkin | |||
| 2015-03-17 09:56 | hgbot | Note Added: 0075681 | |||
| 2015-03-17 09:56 | hgbot | Status | new => resolved | ||
| 2015-03-17 09:56 | hgbot | Resolution | open => fixed | ||
| 2015-03-17 09:56 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/f876c9ef309bcafba6ca8644fc4f01a7462c4c1e [^] | ||
| 2015-03-30 11:45 | marvintm | Note Added: 0076129 | |||
| 2015-03-30 11:45 | marvintm | Status | resolved => new | ||
| 2015-03-30 11:45 | marvintm | Resolution | fixed => open | ||
| 2015-03-30 12:28 | marvintm | Note Deleted: 0076129 | |||
| 2015-03-30 12:35 | marvintm | Status | new => scheduled | ||
| 2015-03-30 12:35 | marvintm | Status | scheduled => resolved | ||
| 2015-03-30 12:35 | marvintm | Fixed in Version | => RR15Q2 | ||
| 2015-03-30 12:35 | marvintm | Resolution | open => fixed | ||
| 2015-03-30 12:38 | marvintm | Review Assigned To | => marvintm | ||
| 2015-03-30 12:38 | marvintm | Status | resolved => closed | ||
| Notes | |||||
|
|
|||||
|
|
||||