From 43b2f11e9b4f42c8b7ce01684ca8f83e4e281156 Mon Sep 17 00:00:00 2001
From: redchris <devnull@localhost>
Date: Tue, 30 Mar 2010 11:51:44 +0000
Subject: [PATCH] BF[ 2945507 ] - Material Cost Element
 http://sourceforge.net/tracker/?func=detail&aid=2945507&group_id=176962&atid=879332
 -- refactoring code for using ADempiere Best Practices Please review

---
 base/src/org/compiere/model/MCostElement.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/base/src/org/compiere/model/MCostElement.java b/base/src/org/compiere/model/MCostElement.java
index acd86806fc..2b53c37c8b 100644
--- a/base/src/org/compiere/model/MCostElement.java
+++ b/base/src/org/compiere/model/MCostElement.java
@@ -92,9 +92,9 @@ public class MCostElement extends X_M_CostElement
 	 */
 	public static MCostElement getMaterialCostElement(Properties ctx, String CostingMethod)
 	{
-		final String whereClause = "CostingMethod=? AND CostElementType=? ";
+		final String whereClause = COLUMNNAME_CostingMethod + " = ? AND " + COLUMNNAME_CostElementType + " = ? ";
 		List<MCostElement> list = new Query(ctx, I_M_CostElement.Table_Name, whereClause, null)
-		.setParameters(CostingMethod,"M")
+		.setParameters(CostingMethod,COSTELEMENTTYPE_Material)
 		.setClient_ID()
 		.setOrderBy(I_M_CostElement.COLUMNNAME_AD_Org_ID)
 		.list();