[jboss-remoting-commits] JBoss Remoting SVN: r4451 - remoting3/trunk/util/src/main/java/org/jboss/cx/remoting/util.

[jboss-remoting-commits at lists.jboss.org]

Author: david.lloyd at jboss.com
Date: 2008-07-31 21:48:05 -0400 (Thu, 31 Jul 2008)
New Revision: 4451

Modified:
   remoting3/trunk/util/src/main/java/org/jboss/cx/remoting/util/ServiceURI.java
Log:
New and improved Remoting service URI utility methods - fixed

Modified: remoting3/trunk/util/src/main/java/org/jboss/cx/remoting/util/ServiceURI.java
===================================================================
--- remoting3/trunk/util/src/main/java/org/jboss/cx/remoting/util/ServiceURI.java	2008-08-01 01:41:51 UTC (rev 4450)
+++ remoting3/trunk/util/src/main/java/org/jboss/cx/remoting/util/ServiceURI.java	2008-08-01 01:48:05 UTC (rev 4451)
@@ -115,14 +115,14 @@
             if (serviceType != null && serviceType.length() > 0) {
                 builder.append(serviceType);
             }
+            builder.append(':');
             if (groupName != null && groupName.length() > 0) {
-                builder.append(':').append(groupName);
-                if (endpointName != null && endpointName.length() > 0) {
-                    builder.append(':').append(groupName);
-                }
-            } else if (endpointName != null && endpointName.length() > 0) {
-                builder.append(':').append(':').append(groupName);
+                builder.append(groupName);
             }
+            builder.append(':');
+            if (endpointName != null && endpointName.length() > 0) {
+                builder.append(endpointName);
+            }
             return new URI(SCHEME, builder.toString(), null);
         } catch (URISyntaxException e) {
             throw new IllegalStateException("URI syntax exception should not be possible here", e);