From 9fb64ab14248f1d8acadfab2536d9c552796a5fb Mon Sep 17 00:00:00 2001
From: teo_sarca <devnull@localhost>
Date: Fri, 6 Feb 2009 14:21:09 +0000
Subject: [PATCH] BF [ 1899477 ] MCash.getLines should return only active lines
 https://sourceforge.net/tracker/?func=detail&atid=879332&aid=1899477&group_id=176962

---
 base/src/org/compiere/model/MCash.java | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/base/src/org/compiere/model/MCash.java b/base/src/org/compiere/model/MCash.java
index 73232f3c64..64b67af633 100644
--- a/base/src/org/compiere/model/MCash.java
+++ b/base/src/org/compiere/model/MCash.java
@@ -45,6 +45,7 @@ import org.compiere.util.TimeUtil;
  *  @author Teo Sarca, SC ARHIPAC SERVICE SRL
  * 			<li>BF [ 1831997 ] Cash journal allocation reversed
  * 			<li>BF [ 1894524 ] Pay an reversed invoice
+ * 			<li>BF [ 1899477 ] MCash.getLines should return only active lines
  */
 public class MCash extends X_C_Cash implements DocAction
 {
@@ -212,12 +213,12 @@ public class MCash extends X_C_Cash implements DocAction
 			return m_lines;
 		}
 		
-		String whereClause ="C_Cash_ID=?"; 
-		List <MCashLine> list = new Query(getCtx(), MCashLine.Table_Name, whereClause, get_TrxName())
-			.setParameters(new Object[]{getC_Cash_ID()})
-			.setOrderBy("Line")
-			.list()
-		;
+		String whereClause =MCashLine.COLUMNNAME_C_Cash_ID+"=?"; 
+		List<MCashLine> list = new Query(getCtx(),MCashLine.Table_Name,  whereClause, get_TrxName())
+								.setParameters(new Object[]{getC_Cash_ID()})
+								.setOrderBy(MCashLine.COLUMNNAME_Line)
+								.setOnlyActiveRecords(true)
+								.list();
 		
 		m_lines =  list.toArray(new MCashLine[list.size()]);
 		return m_lines;