The problem is that for the ajax stuff it needs "buildUrl" to generate the
right url, is there anything we can do to keep them both working?
musachy
On 12/4/06, Don Brown <[EMAIL PROTECTED]> wrote:
Musachy Barroso wrote:
> Before the anchor tag was processing the href attribute like this:
>
> addParameter("href", findString(href));
>
> after the changes for the ajax tags, they are all processing the href
> like this:
>
> addParameter("href", UrlHelper.buildUrl(findString(href), request,
> response, null));
Ok, then that's the problem. When generating a url via the URL tag, it
already calls that buildUrl method, so it shouldn't be called again by
the Anchor tag. We should back that change out as it breaks existing
behavior. In general, please do be careful not to change how a tag
works, as we are going for a fully backwards compatible release in
2.0.x, or at least as close as we can get.
Don
>
> but I can't figure out what is the relationship between Anchor and URL
>
> musachy
>
> Don Brown wrote:
>> From what I can tell, the recent patch WW-1485 broke, or at least
>> changed, the anchor tag. The url is being generated correctly,
>> however, the anchor tag is incorrectly processing the url a second
>> time. Rainer applied the patch, so perhaps he might know how to fix
it?
>>
>> Don
>>
>> Ted Husted wrote:
>>> I just filed
>>>
>>> * https://issues.apache.org/struts/browse/WW-1537
>>>
>>> Links in the showcase that refer to namespaces are rendering the
>>> webapp root twice.
>>>
>>> * http://localhost:8080/struts2-showcase/struts2-showcase/tags/non-ui/
>>> ^^^^ ^^^^
>>> The problem breaks a lot of the showcase links, and is going to break
>>> links in other applications that use namespaces or paths in URL links.
>>>
>>> It might be something simple, but we should fix it before tagging a
>>> build.
>>>
>>> For now, I'm going to look at other things in the Showcase, in the
>>> hope someone has a quick solution to WW-1537.
>>>
>>> -Ted.
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>>> For additional commands, e-mail: [EMAIL PROTECTED]
>>>
>>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
"Hey you! Would you help me to carry the stone?" Pink Floyd
