definitely making progress...

[snip]
> - Removed all query parameter from the method forwardURL(). The
recommended
> way is to use a context tools like Nathan's proposed LinkTool to achieve
this.
> See notes and example at the end of that section. Since the LinkTool makes
> sense with and without Struts, I don't want to add all its functionality
> to the Struts tool(s) but rather keep it as a separate tool.

hmm.

[snip]
> Note:
> If you need to append query data to the URLs created with above methods,
> it is recommended to use a context tool like Nathan's proposed LinkTool.
> Using LinkTool, adding query parameters would works the following way:
>
> #set( $foolink = $link.setPath($struts.forward("login")) )
> <a href="$foolink.addQueryData("action", "checkAuth")">Login</a>

this note is kinda sliding into phase 2 already, but we're pretty close to
that anyway, so i don't think it'll hurt to comment on it...

i don't propose using my LinkTool in any fashion resembling the VTL above.
i think the path, action, forward, query data, etc.  all needs to be part of
the same tool.   the VTL above would become extremely annoying if we had to
do that for every link w/query data.  i would prefer to see a velstruts
specific version of LinkTool  that includes a setAction(String) and
forward(String)  that behave in similar fashion to LinkTool.setPath(String).
that way a link w/query data is as simple as

<a
href="$link.forward('login').addQueryData('action','checkAuth')">Login</a>

it should be simple to modify LinkTool in such a way.  but to talk about
that would be slipping into phase 3....

Nathan Bubna
[EMAIL PROTECTED]



--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to