Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0022805 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
design defect | [Openbravo ERP] 07. Sales management | minor | N/A | 2013-01-16 11:53 | 2013-01-21 12:57 | |||
Reporter | alostale | View Status | public | |||||
Assigned To | alostale | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | 38af5fb36bd9 | ||||
Projection | none | ETA | none | Target Version | 3.0MP20 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | dmiguelez | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0022805: Price Adjustments don't update prices while editing lines | |||||||
Description | Starting from 3.0MP17, existent Price Adjustments were extended in order to add the capability to create more powerful rules to create discounts. Existent Price Adjustments were converted in a type of Promotions and Discounts. Because it is possible to create Promotions and Discounts that are applied based not only in one sales/invoice line but also in the rest of lines in the document, it is not possible anymore to calculate the prices with discounts when creating/editing the line. This is way this calculation was removed and executed only on demand or when posting the document. Though this changes the existent behavior. | |||||||
Steps To Reproduce | 1. Go to Promotions and Discounts 2. Create a new entry that applies to product A (with price list 10€) 3. Discount 10% 4. Create a new Sales Order 5. Add a new line of product A, check Net Unit Price and Line Net Amount are 10€ 6. Book the Order, now Net Unit Price and Line Net Amount are 9€ (10€-10%) Previous to MP17, in step 5 both fields would be 9€ instead of 10€ | |||||||
Proposed Solution | In order to keep backwards compatibility, only for the case of Price Adjustments, final prices will be calculated while the line is being edited. Note that before MP17, Price Adjustment lines were also created just after inserting/editing the line. This is *NOT* going to be restored, so Price Adjustment lines will be only created on posting. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0055539) hgbot (developer) 2013-01-16 11:56 |
Repository: erp/devel/pi Changeset: 38af5fb36bd953a6c62dd057a51c5e66f11b3da3 Author: Asier Lostalé <asier.lostale <at> openbravo.com> Date: Wed Jan 16 11:55:20 2013 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/38af5fb36bd953a6c62dd057a51c5e66f11b3da3 [^] fixed issue 22805: Price Adjustments don't update prices while editing lines --- M src-db/database/model/functions/M_PROMOTION_CALCULATE.xml M src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Amt.java M src/org/openbravo/erpCommon/ad_callouts/SL_Invoice_Product.java M src/org/openbravo/erpCommon/ad_callouts/SL_Order_Amt.java M src/org/openbravo/erpCommon/ad_callouts/SL_Order_Product.java A src/org/openbravo/erpCommon/businessUtility/PriceAdjustment.java --- |
(0055540) alostale (viewer) 2013-01-16 12:09 |
Updated documentation: http://wiki.openbravo.com/wiki/Discounts_and_Promotions [^] |
(0055631) hudsonbot (viewer) 2013-01-18 20:26 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/4cd4a6cc38e0 [^] Maturity status: Test |
(0055670) dmiguelez (viewer) 2013-01-21 12:57 |
Code Review + Testing OK |
![]() |
|||
Date Modified | Username | Field | Change |
2013-01-16 11:53 | alostale | New Issue | |
2013-01-16 11:53 | alostale | Assigned To | => alostale |
2013-01-16 11:53 | alostale | Modules | => Core |
2013-01-16 11:53 | alostale | OBNetwork customer | => No |
2013-01-16 11:53 | alostale | Triggers an Emergency Pack | => No |
2013-01-16 11:54 | alostale | Review Assigned To | => dmiguelez |
2013-01-16 11:56 | hgbot | Checkin | |
2013-01-16 11:56 | hgbot | Note Added: 0055539 | |
2013-01-16 11:56 | hgbot | Status | new => resolved |
2013-01-16 11:56 | hgbot | Resolution | open => fixed |
2013-01-16 11:56 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/38af5fb36bd953a6c62dd057a51c5e66f11b3da3 [^] |
2013-01-16 12:09 | alostale | Note Added: 0055540 | |
2013-01-18 20:26 | hudsonbot | Checkin | |
2013-01-18 20:26 | hudsonbot | Note Added: 0055631 | |
2013-01-21 12:57 | dmiguelez | Note Added: 0055670 | |
2013-01-21 12:57 | dmiguelez | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |