You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ni...@apache.org on 2008/08/06 08:14:15 UTC

svn commit: r683125 - /activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsMessage.java

Author: ningjiang
Date: Tue Aug  5 23:14:15 2008
New Revision: 683125

URL: http://svn.apache.org/viewvc?rev=683125&view=rev
Log:
CAMEL-784 Fixed the NPE issue of JmsMessage

Modified:
    activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsMessage.java

Modified: activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsMessage.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsMessage.java?rev=683125&r1=683124&r2=683125&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsMessage.java (original)
+++ activemq/camel/trunk/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsMessage.java Tue Aug  5 23:14:15 2008
@@ -161,6 +161,10 @@
 
     @Override
     protected String createMessageId() {
+        if (jmsMessage == null) {
+            LOG.info("The jmsMessage is not set yet, call the super's createMessageId");
+            return super.createMessageId();
+        }
         try {
             String id = getDestinationAsString(jmsMessage.getJMSDestination()) + jmsMessage.getJMSMessageID();
             return getSanitizedString(id);