From 69fc191cc7fef69e4d8994aa726c4b43d7ee3e73 Mon Sep 17 00:00:00 2001 From: Heng Sin Low Date: Mon, 26 Mar 2012 14:14:35 +0800 Subject: [PATCH] IDEMPIERE-210 Costing: Revert [ 1693997 ] Calculation of cost should happen when transaction complete. Fixed a regression introduce in previous revision. --- .../src/org/compiere/acct/Doc_InOut.java | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/org.adempiere.base/src/org/compiere/acct/Doc_InOut.java b/org.adempiere.base/src/org/compiere/acct/Doc_InOut.java index 358e78ebf6..7700595f28 100644 --- a/org.adempiere.base/src/org/compiere/acct/Doc_InOut.java +++ b/org.adempiere.base/src/org/compiere/acct/Doc_InOut.java @@ -427,17 +427,19 @@ public class Doc_InOut extends Doc { costs = line.getProductCosts(as, line.getAD_Org_ID(), false); // current costs } + + if (costs == null || costs.signum() == 0) + { + p_Error = "Resubmit - No Costs for " + product.getName(); + log.log(Level.WARNING, p_Error); + return null; + } } else { costs = BigDecimal.ZERO; } - if (costs == null || costs.signum() == 0) - { - p_Error = "Resubmit - No Costs for " + product.getName(); - log.log(Level.WARNING, p_Error); - return null; - } + // Inventory/Asset DR MAccount assets = line.getAccount(ProductCost.ACCTTYPE_P_Asset, as); if (product.isService())