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 | No | ||||
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 | OBNetwork customer | => No | ||
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 | |||||
|
|||||
|
|