You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dw...@apache.org on 2008/05/09 16:13:19 UTC
svn commit: r654820 -
/geronimo/server/branches/2.0/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/AbstractName.java
Author: dwoods
Date: Fri May 9 07:13:19 2008
New Revision: 654820
URL: http://svn.apache.org/viewvc?rev=654820&view=rev
Log:
merged in r654755 from trunk - Include the invalid uri in the exception text
Modified:
geronimo/server/branches/2.0/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/AbstractName.java
Modified: geronimo/server/branches/2.0/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/AbstractName.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/AbstractName.java?rev=654820&r1=654819&r2=654820&view=diff
==============================================================================
--- geronimo/server/branches/2.0/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/AbstractName.java (original)
+++ geronimo/server/branches/2.0/modules/geronimo-kernel/src/main/java/org/apache/geronimo/gbean/AbstractName.java Fri May 9 07:13:19 2008
@@ -118,7 +118,10 @@
//
name = new TreeMap();
String nameString = uri.getQuery();
- if (nameString == null) throw new IllegalArgumentException("uri does not contain a query part used for the name map");
+ if (nameString == null) {
+ throw new IllegalArgumentException("uri does not contain a query part used for the name map; uri: " + uri);
+ }
+
List nameParts = split(nameString, ',');
for (Iterator iterator = nameParts.iterator(); iterator.hasNext();) {
String namePart = (String) iterator.next();