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

BF 2856009: Reversal costing is broken in Physical Inventory 

https://sourceforge.net/tracker/?func=detail&aid=2856009&group_id=176962&atid=879332
This commit is contained in:
armenrz 2009-09-13 10:27:00 +00:00
parent cd6cc98352
commit a7c4fed665
1 changed files with 19 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
base/src/org/compiere/model/MInventory.java
View File

@ -948,11 +948,25 @@ public class MInventory extends X_M_Inventory implements DocAction
continue;
}
ProductCost pc = new ProductCost (Env.getCtx(),
line.getM_Product_ID(), M_AttributeSetInstance_ID, line.get_TrxName());
pc.setQty(qty);
BigDecimal costs = pc.getProductCosts(as, line.getAD_Org_ID(), as.getCostingMethod(),
0,false);
BigDecimal costs = Env.ZERO;
if (isReversal())
{
String sql = "SELECT amt * -1 FROM M_CostDetail WHERE M_InventoryLine_ID=?"; // negate costs
MProduct product = new MProduct(getCtx(), line.getM_Product_ID(), line.get_TrxName());
String CostingLevel = product.getCostingLevel(as);
if (MAcctSchema.COSTINGLEVEL_Organization.equals(CostingLevel))
sql = sql + " AND AD_Org_ID=" + getAD_Org_ID();
else if (MAcctSchema.COSTINGLEVEL_BatchLot.equals(CostingLevel) && M_AttributeSetInstance_ID != 0)
sql = sql + " AND M_AttributeSetInstance_ID=" + M_AttributeSetInstance_ID;
costs = DB.getSQLValueBD(line.get_TrxName(), sql, line.getReversalLine_ID());
}
else
{
ProductCost pc = new ProductCost (getCtx(),
line.getM_Product_ID(), M_AttributeSetInstance_ID, line.get_TrxName());
pc.setQty(qty);
costs = pc.getProductCosts(as, line.getAD_Org_ID(), as.getCostingMethod(), 0,false);
}
if (costs == null || costs.signum() == 0)
{
return "No Costs for " + line.getProduct().getName();