Re: [tools] StrutsLinkTool's setForward method has different behavior

All,

On 7/28/2011 12:48 PM, Christopher Schultz wrote:
> I think the solution is to modify this so that
> generic.LinkTool.absolute() /does/ use the URI class to parse the URI
> and separate-out the query string, etc. and then set them on the copied
> LinkTool's instance.

Okay, this works, but I'm not sure if it's the best solution, or if it's
even a complete solution. StrutsLinkTool calls LinkTool.absolute(), but
the path is actually a relative path, so maybe calling
LinkTool.relative() is a better thing to do. In that case, the fix
should probably be different.

Quick diff for generic.LinkTool:


Index: src/main/java/org/apache/velocity/tools/generic/LinkTool.java
===================================================================
--- src/main/java/org/apache/velocity/tools/generic/LinkTool.java
(revision 1151249)
+++ src/main/java/org/apache/velocity/tools/generic/LinkTool.java
(working copy)
@@ -1357,6 +1357,7 @@
                 {
                     return null;
                 }
+
                 copy.setScheme(uri.getScheme());
                 copy.setUserInfo(uri.getUserInfo());
                 copy.setHost(uri.getHost());
@@ -1378,8 +1379,30 @@
                 }
                 return copy;
             }
-            else if (!pth.startsWith("/"))
+            else if (pth.startsWith("/"))
             {
+                // This is a relative path, not absolute.
+                // That's okay -- parse the URI anyway but don't process
+                // the host, scheme, port, user, etc.
+                URI uri = toURI(pth);
+
+                pth = uri.getPath();
+                if (pth.equals("/") || pth.length() == 0)
+                {
+                    pth = null;
+                }
+                copy.setPath(pth);
+                if (uri.getQuery() != null)
+                {
+                    copy.setQuery(uri.getQuery());
+                }
+                if (uri.getFragment() != null)
+                {
+                    copy.setFragment(uri.getFragment());
+                }
+            }
+            else
+            {
                 // paths that don't start with '/'
                 // are considered relative to the current directory
                 pth = combinePath(getDirectory(), pth);