[jbpm-commits] JBoss JBPM SVN: r4468 - jbpm4/branches/email/modules/pvm/src/main/java/org/jbpm/pvm/internal/email/service.

[do-not-reply at jboss.org]

Author: bradsdavis
Date: 2009-04-06 20:33:23 -0400 (Mon, 06 Apr 2009)
New Revision: 4468

Modified:
   jbpm4/branches/email/modules/pvm/src/main/java/org/jbpm/pvm/internal/email/service/SynchronousMailService.java
Log:
Added logger, todo.

Modified: jbpm4/branches/email/modules/pvm/src/main/java/org/jbpm/pvm/internal/email/service/SynchronousMailService.java
===================================================================
--- jbpm4/branches/email/modules/pvm/src/main/java/org/jbpm/pvm/internal/email/service/SynchronousMailService.java	2009-04-07 00:29:17 UTC (rev 4467)
+++ jbpm4/branches/email/modules/pvm/src/main/java/org/jbpm/pvm/internal/email/service/SynchronousMailService.java	2009-04-07 00:33:23 UTC (rev 4468)
@@ -11,6 +11,8 @@
 import javax.mail.internet.MimeMessage;
 import javax.mail.internet.MimeMessage.RecipientType;
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.apache.commons.mail.Email;
 import org.apache.commons.mail.EmailException;
 import org.jbpm.pvm.internal.email.filter.AddressFilter;
@@ -19,6 +21,8 @@
 
 public class SynchronousMailService implements MailService {
 
+	private static final Log log = LogFactory.getLog(SynchronousMailService.class);
+	
 	Collection<MailServerConfiguration> serverConfigs;
 	
 	public void send(Collection<Email> emails) {
@@ -61,7 +65,8 @@
 					}
 					catch(EmailException e)
 					{
-						//TODO: What to do here?
+						log.error("An exception occurred while sending email.",e);
+						//TODO: Does anything else need to occur here?
 					}
 				}
 			}