From 12ea725d2b2cd95e9a8ef5f4b4511e841054b628 Mon Sep 17 00:00:00 2001 From: Rajith Muditha Attapattu Date: Wed, 27 Feb 2008 20:01:12 +0000 Subject: Fixed the toString method to avoid printing both a routingkey and bindingkey and also to properly printout the bindingkeys when required. This bug caused 3 test failures. git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@631698 13f79535-47bb-0310-9956-ffa450edef68 --- .../main/java/org/apache/qpid/client/AMQDestination.java | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) (limited to 'java/client/src') diff --git a/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java b/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java index 0cf1cb32a1..e02b3d6643 100644 --- a/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java +++ b/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java @@ -274,19 +274,19 @@ public abstract class AMQDestination implements Destination, Referenceable sb.append(URLHelper.DEFAULT_OPTION_SEPERATOR); } - if (_bindingKeys != null && _bindingKeys.length>0) + // We can't allow both routingKey and bindingKey + if (_routingKey == null && _bindingKeys != null && _bindingKeys.length>0) { - sb.append(BindingURL.OPTION_BINDING_KEY); - sb.append("='"); + for (AMQShortString bindingKey:_bindingKeys) { - - sb.append(bindingKey).append(","); + sb.append(BindingURL.OPTION_BINDING_KEY); + sb.append("='"); + sb.append(bindingKey); + sb.append("'"); + sb.append(URLHelper.DEFAULT_OPTION_SEPERATOR); } - sb.deleteCharAt(sb.length() - 1); - sb.append("'"); - sb.append(URLHelper.DEFAULT_OPTION_SEPERATOR); } if (_isDurable) -- cgit v1.2.1