midsuit
/
core-jgi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

IDEMPIERE-190 Average Costing: make sure amt and qty argument for the setWeightedAverage method is of the same sign

This commit is contained in:
Carlos Ruiz 2013-07-03 21:41:54 -05:00
parent 1727029c25
commit dad7803507
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
org.adempiere.base/src/org/compiere/model/MCost.java
View File

@ -1482,9 +1482,9 @@ public class MCost extends X_M_Cost
public void setWeightedAverage (BigDecimal amt, BigDecimal qty) public void setWeightedAverage (BigDecimal amt, BigDecimal qty)
{ {
//amount must follow the sign of qty //amount must follow the sign of qty
if (amt.signum() != 0 && amt.signum() != qty.signum()) if (amt.signum() != 0 && qty.signum() != 0 && amt.signum() != qty.signum())
{ {
amt = amt.multiply(BigDecimal.valueOf(-1.00d)); amt = amt.negate();
} }
if (getCurrentQty().add(qty).signum() < 0) if (getCurrentQty().add(qty).signum() < 0)