You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by no...@apache.org on 2011/08/10 14:56:42 UTC
svn commit: r1156159 -
/james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java
Author: norman
Date: Wed Aug 10 12:56:42 2011
New Revision: 1156159
URL: http://svn.apache.org/viewvc?rev=1156159&view=rev
Log:
Only log error on non null values. See JAMES-1294
Modified:
james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java
Modified: james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java
URL: http://svn.apache.org/viewvc/james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java?rev=1156159&r1=1156158&r2=1156159&view=diff
==============================================================================
--- james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java (original)
+++ james/server/trunk/queue-jms/src/main/java/org/apache/james/queue/jms/JMSMailQueue.java Wed Aug 10 12:56:42 2011
@@ -423,10 +423,14 @@ public class JMSMailQueue implements Man
// Now cast the property back to Serializable and set it as attribute.
// See JAMES-1241
Object attrValue = message.getObjectProperty(name);
- if ( attrValue instanceof Serializable) {
- mail.setAttribute(name, (Serializable) attrValue);
- } else {
- logger.error("Not supported mail attribute " + name + " of type " + attrValue);
+
+ // ignore null values. See JAMES-1294
+ if (attrValue != null) {
+ if (attrValue instanceof Serializable) {
+ mail.setAttribute(name, (Serializable) attrValue);
+ } else {
+ logger.error("Not supported mail attribute " + name + " of type " + attrValue + " for mail " + mail.getName());
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org