From 3ec5707657be7509752ec3a117c29c0a6ba01cbc Mon Sep 17 00:00:00 2001
From: Heng Sin Low <hengsin@gmail.com>
Date: Fri, 9 Mar 2012 10:29:00 +0800
Subject: [PATCH] IDEMPIERE-190 Average Costing: make sure amt and qty argument
 for the setWeightedAverage method is of the same sight

---
 org.adempiere.base/src/org/compiere/model/MCost.java | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/org.adempiere.base/src/org/compiere/model/MCost.java b/org.adempiere.base/src/org/compiere/model/MCost.java
index bb7e41992d..9b53af0cd7 100644
--- a/org.adempiere.base/src/org/compiere/model/MCost.java
+++ b/org.adempiere.base/src/org/compiere/model/MCost.java
@@ -1453,6 +1453,11 @@ public class MCost extends X_M_Cost
 	 */
 	public void setWeightedAverage (BigDecimal amt, BigDecimal qty)
 	{
+		//amount must follow the sign of qty
+		if (amt.signum() != 0 && amt.signum() != qty.signum())
+		{
+			amt = amt.multiply(new BigDecimal(-1.00d));
+		}
 		BigDecimal oldSum = getCurrentCostPrice().multiply(getCurrentQty());
 		BigDecimal newSum = amt;	//	is total already
 		BigDecimal sumAmt = oldSum.add(newSum);