svn commit: r594794 - /ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/finaccount/FinAccountServices.java

[jaz]

Author: jaz
Date: Wed Nov 14 00:02:02 2007
New Revision: 594794

URL: http://svn.apache.org/viewvc?rev=594794&view=rev
Log:
fixed bug in fin account refunds which refunded transactions not created by the 
account being refunded

Modified:
    
ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/finaccount/FinAccountServices.java

Modified: 
ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/finaccount/FinAccountServices.java
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/finaccount/FinAccountServices.java?rev=594794&r1=594793&r2=594794&view=diff
==============================================================================
--- 
ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/finaccount/FinAccountServices.java
 (original)
+++ 
ofbiz/trunk/applications/accounting/src/org/ofbiz/accounting/finaccount/FinAccountServices.java
 Wed Nov 14 00:02:02 2007
@@ -19,28 +19,25 @@
 
 package org.ofbiz.accounting.finaccount;
 
+import java.math.BigDecimal;
 import java.sql.Timestamp;
-import java.util.Map;
 import java.util.List;
-import java.math.BigDecimal;
+import java.util.Map;
 
 import org.ofbiz.base.util.*;
 import org.ofbiz.entity.GenericDelegator;
 import org.ofbiz.entity.GenericEntityException;
 import org.ofbiz.entity.GenericValue;
+import org.ofbiz.entity.condition.EntityCondition;
+import org.ofbiz.entity.condition.EntityConditionList;
 import org.ofbiz.entity.condition.EntityExpr;
 import org.ofbiz.entity.condition.EntityOperator;
-import org.ofbiz.entity.condition.EntityCondition;
 import org.ofbiz.entity.util.EntityListIterator;
 import org.ofbiz.entity.util.EntityUtil;
-import org.ofbiz.service.DispatchContext;
-import org.ofbiz.service.GenericServiceException;
-import org.ofbiz.service.LocalDispatcher;
-import org.ofbiz.service.ModelService;
-import org.ofbiz.service.ServiceUtil;
-
 import org.ofbiz.order.finaccount.FinAccountHelper;
 import org.ofbiz.product.store.ProductStoreWorker;
+import org.ofbiz.service.*;
+
 import javolution.util.FastMap;
 
 public class FinAccountServices {
@@ -352,7 +349,10 @@
                 BigDecimal remainingBalance = new 
BigDecimal(actualBalance.toString());
                 BigDecimal refundAmount = BigDecimal.ZERO;
 
-                EntityCondition condition = new 
EntityExpr("finAccountTransTypeId", EntityOperator.EQUALS, "DEPOSIT");
+                List exprs = UtilMisc.toList(new 
EntityExpr("finAccountTransTypeId", EntityOperator.EQUALS, "DEPOSIT"),
+                        new EntityExpr("finAccountId", EntityOperator.EQUALS, 
finAccountId));
+                EntityCondition condition = new EntityConditionList(exprs, 
EntityOperator.AND);
+
                 EntityListIterator eli = null;
                 try {
                     eli = 
delegator.findListIteratorByCondition("FinAccountTrans", condition, null, 
UtilMisc.toList("-transactionDate"));