[jira] Resolved: (WICKET-2050) KittenCaptchaPanel generates a Ajax callback image url which potentially breaks when used in a Portlet

["Ate Douma (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/WICKET-2050?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ate Douma resolved WICKET-2050.
-------------------------------

    Resolution: Fixed

Fixed

> KittenCaptchaPanel generates a Ajax callback image url which potentially breaks when used in a Portlet   
> ---------------------------------------------------------------------------------------------------------
>
>                 Key: WICKET-2050
>                 URL: https://issues.apache.org/jira/browse/WICKET-2050
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket-extensions
>    Affects Versions: 1.4-RC1
>            Reporter: Ate Douma
>            Assignee: Ate Douma
>            Priority: Minor
>             Fix For: 1.4-RC2
>
>
> KittenCaptchaPanel generates the Ajax callback image url in its onclick event handler using:
> 	javascript.append("Wicket.$('" + image.getMarkupId() + "').src = '");
> 	javascript.append(image.urlFor(IResourceListener.INTERFACE));
> 	javascript.append("&rand=" + Math.random());
> 	javascript.append("'");
> Appending the rand parameter using "&rand=" assumes the image url already contains (at least) one other parameter.
> Within a default (plain website) environment this most likely is true, however in a Portlet this really depends on how the portal encodes the generated image url in the resulting PortletURL.
> Jetspeed-2 for instance (can) encode this as path element, in which case appending "&rand=" leads to an incorrect and invalid url.
> For further information, please see WICKET-649 in which I described how I fixed similar cases in several other locations within Wicket.
> I'll fix this issue likewise in a transparent way replacing the above with the following:
> 	javascript.append("Wicket.$('" + image.getMarkupId() + "').src = '");
> 	CharSequence url = image.urlFor(IResourceListener.INTERFACE);
> 	javascript.append(url);
> 	javascript.append((url.toString().indexOf('?') > -1 ? "&" : "?") + "rand=" + Math.random());
> 	javascript.append("'");

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