Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0006559Openbravo ERPZ. Otherspublic2008-12-15 17:242009-03-03 17:16
shuehner 
AinhoaPagola 
normalmajorhave not tried
closedfixed 
5
pi 
 
Core
No
0006559: SL_Order_Amt callout does not round all price fields but only some
This trigger does round the two fields: priceActual and priceLimit according to the selected PricePrevision (setup in the currency). It does not round the fileds priceList and priceStd.

This leads to strange/wrong behavior when a price is entered which will be rounded to a different value.

The number of decimal places for the prices has to be chosen according to the configured PricePrecision in the currency setup. For the EUR currency the price precision is 4 in r2.40 and 3 in r2.3x so a value with at least 5 digits after the comma has to be chosen to get a price which is rounded to a different value in both openbravo versions.
Create a Purchase Order
Create a new line
Select i.e. the beer product
Enter: 0.00008 as unit price
Enter: 0.00008 as List Price
See that the discount is calculated as 25% which is not correct
Review the SL_Order_Amt trigger according to rounding.
comsup_sprint4
blocks defect 0006558 closed AinhoaPagola SL_Order_Amt callout does not round all price fields but only some 
Issue History
2008-12-15 17:24shuehnerNew Issue
2008-12-15 17:24shuehnerAssigned To => rafaroda
2008-12-15 17:24shuehnerStatusnew => scheduled
2009-01-09 20:10rafarodaAssigned Torafaroda => AinhoaPagola
2009-01-13 19:29svnbotCheckin
2009-01-13 19:29svnbotNote Added: 0012156
2009-01-13 19:29svnbotStatusscheduled => resolved
2009-01-13 19:29svnbotResolutionopen => fixed
2009-01-13 19:29svnbotsvn_revision => 11855
2009-01-23 11:39AinhoaPagolaTag Attached: comsup_sprint4
2009-03-03 17:16psarobeStatusresolved => closed

Notes
(0012156)
svnbot   
2009-01-13 19:29   
Repository: openbravo
Revision: 11855
Author: AinhoaPagola
Date: 2009-01-13 19:29:16 +0100 (Tue, 13 Jan 2009)

Fixes bug 0006559 SL_order_amt callout rounds prices to the same precision.

---
U branches/r2.40/src/org/openbravo/erpCommon/ad_callouts/SL_Order_Amt.java
---

https://dev.openbravo.com/websvn/openbravo/?rev=11855&sc=1 [^]