From d3741a31b20d84e4ebcf5497f7731e12a92cd766 Mon Sep 17 00:00:00 2001
From: Carlos Ruiz <carg67@gmail.com>
Date: Wed, 17 Mar 2010 19:09:57 +0000
Subject: [PATCH] Peer review revision 11673, 11675, 11688 - some refactoring
 Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883

---
 .../org/compiere/model/MPriceListVersion.java | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/base/src/org/compiere/model/MPriceListVersion.java b/base/src/org/compiere/model/MPriceListVersion.java
index 90633538a0..883848fd83 100644
--- a/base/src/org/compiere/model/MPriceListVersion.java
+++ b/base/src/org/compiere/model/MPriceListVersion.java
@@ -108,18 +108,17 @@ public class MPriceListVersion extends X_M_PriceList_Version
 	
 	/**
 	 * 	Get Product Price
-	 *	@return product price
+	 * 	@param whereClause optional where clause
+	 * 	@return product price
 	 */
-	public MProductPrice[] getProductPrice (String passedClause)
+	public MProductPrice[] getProductPrice (String whereClause)
 	{
-		if (passedClause != null)
-			passedClause=" "+passedClause;
-		else 
-			passedClause="";
-		final String whereClause = I_M_ProductPrice.COLUMNNAME_M_PriceList_Version_ID+"=?"+passedClause;
-		List<MProductPrice> list = new Query(getCtx(),I_M_ProductPrice.Table_Name,whereClause,get_TrxName())
-		.setParameters(getM_PriceList_Version_ID())
-		.list();
+		String localWhereClause = I_M_ProductPrice.COLUMNNAME_M_PriceList_Version_ID+"=?"+whereClause;
+		if (whereClause != null)
+			localWhereClause += " " + whereClause;
+		List<MProductPrice> list = new Query(getCtx(),I_M_ProductPrice.Table_Name,localWhereClause,get_TrxName())
+			.setParameters(getM_PriceList_Version_ID())
+			.list();
 		MProductPrice[] pp = new MProductPrice[list.size()];
 		list.toArray(pp);
 		return pp;