You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ro...@apache.org on 2010/10/11 19:38:48 UTC

svn commit: r1021441 - /qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageTransferHeader.java

Author: robbie
Date: Mon Oct 11 17:38:48 2010
New Revision: 1021441

URL: http://svn.apache.org/viewvc?rev=1021441&view=rev
Log:
QPID-2898: when UUID is null, return null instead of doing String.valueOf()

Modified:
    qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageTransferHeader.java

Modified: qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageTransferHeader.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageTransferHeader.java?rev=1021441&r1=1021440&r2=1021441&view=diff
==============================================================================
--- qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageTransferHeader.java (original)
+++ qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageTransferHeader.java Mon Oct 11 17:38:48 2010
@@ -26,6 +26,7 @@ import org.apache.qpid.transport.Message
 
 import java.util.Set;
 import java.util.Map;
+import java.util.UUID;
 
 class MessageTransferHeader implements AMQMessageHeader
 {
@@ -61,7 +62,9 @@ class MessageTransferHeader implements A
 
     public String getMessageId()
     {
-        return _messageProps == null ? null : String.valueOf(_messageProps.getMessageId());
+        UUID id = _messageProps == null ? null : _messageProps.getMessageId();
+        
+        return id == null ? null : String.valueOf(id);
     }
 
     public String getMimeType()



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org