[tinyTiM-commits] SF.net SVN: tinytim:[413] tinytim/trunk

[<lh...@us...>]

Revision: 413
          http://tinytim.svn.sourceforge.net/tinytim/?rev=413&view=rev
Author:   lheuer
Date:     2010-09-03 16:01:18 +0000 (Fri, 03 Sep 2010)

Log Message:
-----------
Resolving references with whitespaces did not work. Fixed.

Modified Paths:
--------------
    tinytim/trunk/CHANGES.txt
    tinytim/trunk/src/main/java/org/tinytim/core/value/LocatorImpl.java
    tinytim/trunk/src/test/java/org/tinytim/core/value/TestLocatorImpl.java

Modified: tinytim/trunk/CHANGES.txt
===================================================================
--- tinytim/trunk/CHANGES.txt	2010-09-02 10:33:10 UTC (rev 412)
+++ tinytim/trunk/CHANGES.txt	2010-09-03 16:01:18 UTC (rev 413)
@@ -9,9 +9,11 @@
   (contributed by Hannes Niederhausen)
 * Removed String-Interning for object ids 
   (caused heap memory problems and fixes a bug reported by Uta Schulze)
-  
+* Locator: Resolving a reference with white spaces caused problems. Fixed.
+  (reported by Uta Schulze)
 
 
+
 2.0.0 (2010-03-19)
 ------------------
 * Updated TMAPI to 2.0.2 (kept compatibility to 2.0 and 2.0.1)

Modified: tinytim/trunk/src/main/java/org/tinytim/core/value/LocatorImpl.java
===================================================================
--- tinytim/trunk/src/main/java/org/tinytim/core/value/LocatorImpl.java	2010-09-02 10:33:10 UTC (rev 412)
+++ tinytim/trunk/src/main/java/org/tinytim/core/value/LocatorImpl.java	2010-09-03 16:01:18 UTC (rev 413)
@@ -148,7 +148,7 @@
         if (_EMPTY.equals(reference)) {
             return this;
         }
-        return create(_uri.resolve(reference));
+        return create(_uri.resolve(reference.replace(" ", "%20")));
     }
 
     /* (non-Javadoc)

Modified: tinytim/trunk/src/test/java/org/tinytim/core/value/TestLocatorImpl.java
===================================================================
--- tinytim/trunk/src/test/java/org/tinytim/core/value/TestLocatorImpl.java	2010-09-02 10:33:10 UTC (rev 412)
+++ tinytim/trunk/src/test/java/org/tinytim/core/value/TestLocatorImpl.java	2010-09-03 16:01:18 UTC (rev 413)
@@ -38,6 +38,11 @@
         assertEquals("http://www.example.org/test%20me/", loc3.toExternalForm());
     }
 
+    public void testResolve() {
+        Locator loc = _tm.createLocator("http://www.example.org/");
+        assertEquals("http://www.example.org/#uta%20schulze", loc.resolve("#uta schulze").toExternalForm());
+    }
+
 //    public void testLowerCaseScheme() {
 //        Locator loc = _tm.createLocator("HTTP://www.example.org/test+me/");
 //        assertEquals("http://www.example.org/test me/", loc.getReference());


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.