svn commit: r1173520 - /ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1173520 - /ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java

sascharodekamp
Author: sascharodekamp
Date: Wed Sep 21 07:40:46 2011
New Revision: 1173520

URL: http://svn.apache.org/viewvc?rev=1173520&view=rev
Log:
EntityListIterator not being closed after storing incoming email (https://issues.apache.org/jira/browse/OFBIZ-4424). A slightly modified  patch from Varun Bhansaly which closes the entity Iterator after use: CommunicationEventServices.buildListOfWorkEffortInfoFromEmailAddresses() does not close the entity list iterator resulting in error messages.

Modified:
    ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java

Modified: ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java?rev=1173520&r1=1173519&r2=1173520&view=diff
==============================================================================
--- ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java (original)
+++ ofbiz/branches/release11.04/applications/party/src/org/ofbiz/party/communication/CommunicationEventServices.java Wed Sep 21 07:40:46 2011
@@ -1149,6 +1149,12 @@ public class CommunicationEventServices
                             tempResults.addAll(filteredList);
                         } catch (GenericEntityException e) {
                             Debug.logError(e, module);
+                        } finally {
+                            try {
+                                listIt.close();
+                            } catch (GenericEntityException e) {
+                                Debug.logError(e, module);
+                            }
                         }
                     }
                 }