You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@ofbiz.apache.org by as...@apache.org on 2010/12/28 10:44:17 UTC

svn commit: r1053289 - /ofbiz/trunk/framework/common/src/org/ofbiz/common/email/EmailServices.java

Author: ashish
Date: Tue Dec 28 09:44:17 2010
New Revision: 1053289

URL: http://svn.apache.org/viewvc?rev=1053289&view=rev
Log:
Bug fix.
When there is failure in sending email from OFBiz we need to break the loop.

Thanks Pranay for the contribution. 

Modified:
    ofbiz/trunk/framework/common/src/org/ofbiz/common/email/EmailServices.java

Modified: ofbiz/trunk/framework/common/src/org/ofbiz/common/email/EmailServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/src/org/ofbiz/common/email/EmailServices.java?rev=1053289&r1=1053288&r2=1053289&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/src/org/ofbiz/common/email/EmailServices.java (original)
+++ ofbiz/trunk/framework/common/src/org/ofbiz/common/email/EmailServices.java Tue Dec 28 09:44:17 2010
@@ -330,6 +330,7 @@ public class EmailServices {
                     SMTPAddressFailedException safe = (SMTPAddressFailedException) nestedException;
                     Debug.logError("Failed to send message to [" + safe.getAddress() + "], return code [" + safe.getReturnCode() + "], return message [" + safe.getMessage() + "]", errMsg);
                     failedAddresses.add(safe);
+                    break;
                 }
             }
             Boolean sendFailureNotification = (Boolean) context.get("sendFailureNotification");