You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ta...@apache.org on 2010/11/09 17:27:03 UTC
svn commit: r1033064 - in
/activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol:
StompFrame.cs StompWireFormat.cs
Author: tabish
Date: Tue Nov 9 16:27:03 2010
New Revision: 1033064
URL: http://svn.apache.org/viewvc?rev=1033064&view=rev
Log:
Fix the Stomp headers read code to check for the correct thing, should be key, was checking value. This broker a unit test. Remove an unnecessary trace statement.
Modified:
activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompFrame.cs
activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompWireFormat.cs
Modified: activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompFrame.cs
URL: http://svn.apache.org/viewvc/activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompFrame.cs?rev=1033064&r1=1033063&r2=1033064&view=diff
==============================================================================
--- activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompFrame.cs (original)
+++ activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompFrame.cs Tue Nov 9 16:27:03 2010
@@ -203,7 +203,7 @@ namespace Apache.NMS.Stomp.Protocol
// Stomp v1.1+ allows multiple copies of a property, the first
// one is considered to be the newest, we could figure out how
// to store them all but for now we just throw the rest out.
- if(!this.properties.Contains(value))
+ if(!this.properties.Contains(key))
{
this.properties[key] = value;
}
Modified: activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompWireFormat.cs
URL: http://svn.apache.org/viewvc/activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompWireFormat.cs?rev=1033064&r1=1033063&r2=1033064&view=diff
==============================================================================
--- activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompWireFormat.cs (original)
+++ activemq/activemq-dotnet/Apache.NMS.Stomp/trunk/src/main/csharp/Protocol/StompWireFormat.cs Tue Nov 9 16:27:03 2010
@@ -264,9 +264,6 @@ namespace Apache.NMS.Stomp.Protocol
message.ReplyTo = Destination.ConvertToDestination(frame.RemoveProperty("reply-to"));
message.TargetConsumerId = new ConsumerId(frame.RemoveProperty("subscription"));
message.CorrelationId = frame.RemoveProperty("correlation-id");
-
- Tracer.Debug("RECV - Inbound MessageId = " + frame.GetProperty("message-id"));
-
message.MessageId = new MessageId(frame.RemoveProperty("message-id"));
message.Persistent = StompHelper.ToBool(frame.RemoveProperty("persistent"), false);