Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0006558Openbravo ERPZ. Otherspublic2008-12-15 17:242009-04-21 12:36
Reportershuehner 
Assigned ToAinhoaPagola 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0006558: SL_Order_Amt callout does not round all price fields but only some
DescriptionThis 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.
Steps To ReproduceCreate 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
Proposed SolutionReview the SL_Order_Amt callout according to rounding.
Additional Information
Tagscomsup_sprint4
Relationships
related to backport 0006546 closed rafaroda Format.xml not correctly taken 
depends on backport 0006559 closed AinhoaPagola SL_Order_Amt callout does not round all price fields but only some 
depends on backport 0006560 closed AinhoaPagola SL_Order_Amt callout does not round all price fields but only some 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-12-15 17:24shuehnerNew Issue
2008-12-15 17:24shuehnerAssigned To => rafaroda
2008-12-15 17:24shuehnersf_bug_id0 => 2431245
2008-12-15 17:24shuehnerRegression testing => No
2008-12-15 17:24shuehnerStatusnew => scheduled
2008-12-15 17:24shuehnerfix_in_branch => trunk
2008-12-15 17:24shuehnerRelationship addedrelated to 0006546
2008-12-15 17:25shuehnerNote Added: 0011378
2009-01-09 20:10rafarodaAssigned Torafaroda => AinhoaPagola
2009-01-13 18:49AinhoaPagolafix_in_branchtrunk =>
2009-01-13 18:49AinhoaPagolaSteps to Reproduce Updated
2009-01-13 19:22svnbotCheckin
2009-01-13 19:22svnbotNote Added: 0012155
2009-01-13 19:22svnbotStatusscheduled => resolved
2009-01-13 19:22svnbotResolutionopen => fixed
2009-01-13 19:22svnbotsvn_revision => 11854
2009-01-23 11:39AinhoaPagolaTag Attached: comsup_sprint4
2009-04-21 12:36psarobeStatusresolved => closed

Notes
(0011378)
shuehner   
2008-12-15 17:25   
The related issue 6546 against r2.3x seems to be a special case of this defect.
(0012155)
svnbot   
2009-01-13 19:22   
Repository: openbravo
Revision: 11854
Author: AinhoaPagola
Date: 2009-01-13 19:22:36 +0100 (Tue, 13 Jan 2009)

Fixes bug 0006558 SL_Order_Amt callout rounds all prices to the same precision

---
U trunk/src/org/openbravo/erpCommon/ad_callouts/SL_Order_Amt.java
---

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