[ 
https://issues.apache.org/jira/browse/WICKET-1149?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12553713
 ] 

Doug Leeper commented on WICKET-1149:
-------------------------------------

The second URL is what I am seeing generated.  I am not doing it...it is the 
result of the current code.

I currently have an location implementation of LocalizedResourceReference in my 
project.  If I take out my changes (so it is the same in the TRUNK), I get the 
& added in the URL.  To get my attributes that I have added, i.e. 
"maxHeight",  I have to use the parameter name "amp;maxHeight".

The code Strings.replaceAll(url, "&", "&")));  is keeping the & and adding 
amp;

I understand what you are doing...but it is not necessary as by suggested fix 
and my use locally.

> LocalizedImageResource not formatting parameters properly when using 
> ResourceReference
> --------------------------------------------------------------------------------------
>
>                 Key: WICKET-1149
>                 URL: https://issues.apache.org/jira/browse/WICKET-1149
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.3.0-beta4, 1.3.0-rc1
>            Reporter: Doug Leeper
>            Assignee: Matej Knopp
>            Priority: Critical
>             Fix For: 1.3.0-rc3
>
>
> When using a ResourceReference, the resultant URL generated is not formatted 
> properly when using more than one parameter.  
> For example:
> Correct URL:
> http://localhost:8081/homeIQ/app/resources/homeiq.wicket.HomeIQApplication/modelPicture?thumbnail=true&maxHeight=100&maxWidth=100&id=1
> Incorrect URL: (generated by LocalizedResourceReference)
> http://localhost:8081/homeIQ/app/resources/homeiq.wicket.HomeIQApplication/modelPicture?thumbnail=true&maxHeight=100&maxWidth=100&id=1
> As can be seen, the & has been replaced with &.  This is the result of 
> the following code:
>               tag.put("src", 
> RequestCycle.get().getOriginalResponse().encodeURL(
>                       Strings.replaceAll(url, "&", "&")));
> It should be:
>               tag.put("src", 
> RequestCycle.get().getOriginalResponse().encodeURL(url);

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to