From fb6232be3507fbcb1ac9c20f43dd8255ec367792 Mon Sep 17 00:00:00 2001
From: armenrz <devnull@localhost>
Date: Fri, 10 Oct 2008 09:45:55 +0000
Subject: [PATCH] BF [ 1693997 ] Calculation of cost should happen when
 transaction complete. Fixed wrong location.

---
 base/src/org/compiere/model/MMatchPO.java | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/base/src/org/compiere/model/MMatchPO.java b/base/src/org/compiere/model/MMatchPO.java
index 08e05dcb64..49e5a09761 100644
--- a/base/src/org/compiere/model/MMatchPO.java
+++ b/base/src/org/compiere/model/MMatchPO.java
@@ -579,6 +579,16 @@ public class MMatchPO extends X_M_MatchPO
 	 */
 	protected boolean afterSave (boolean newRecord, boolean success)
 	{
+		if (newRecord && success)
+		{	
+			// Elaine 2008/6/20	
+			String err = createMatchPOCostDetail(this, getOrderLine());
+			if(err != null && err.length() > 0) 
+			{
+				s_log.warning(err);
+				return false;
+			}
+		}
 		//	Purchase Order Delivered/Invoiced
 		//	(Reserved in VMatch and MInOut.completeIt)
 		if (success && getC_OrderLine_ID() != 0)
@@ -613,16 +623,6 @@ public class MMatchPO extends X_M_MatchPO
 			return orderLine.save();
 		}
 		//
-		if (newRecord)
-		{	
-			// Elaine 2008/6/20	
-			String err = createMatchPOCostDetail(this, getOrderLine());
-			if(err != null && err.length() > 0) 
-			{
-				s_log.warning(err);
-				return false;
-			}
-		}
 		return success;
 	}	//	afterSave