You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Martin Grigorov (Resolved) (JIRA)" <ji...@apache.org> on 2011/11/01 16:13:32 UTC

[jira] [Resolved] (WICKET-3956) (Indexed|MixedParam)HybridUrlCodingStrategy ignore interface parameter

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

Martin Grigorov resolved WICKET-3956.
-------------------------------------

    Resolution: Won't Fix

The changes in the produced URLs look too dangerous to be applied that late in 1.4.x which is close to end of life.
The UrlCodingStrategies are reworked in 1.5.x and this problem should be fixed there.
I recommend you to upgrade to 1.5.x.
                
> (Indexed|MixedParam)HybridUrlCodingStrategy ignore interface parameter
> ----------------------------------------------------------------------
>
>                 Key: WICKET-3956
>                 URL: https://issues.apache.org/jira/browse/WICKET-3956
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.4.18
>            Reporter: Dan Retzlaff
>         Attachments: wicket-3956.patch
>
>
> Whereas BookmarkablePageRequestTargetUrlCodingStrategy looks to both the decoded parameters *and* the request parameters for an interface, HybridUrlCodingStrategy does not. This means its MixedParam and Indexed subclasses, for which not all parameters are encoded in the URL, get decoded into HybridBookmarkablePageRequestTargets instead of BookmarkableListenerInterfaceRequestTarget. In my application, this causes form submits to break.
> The attached patch modifies testStatelessHybridUrlCodingStrategy to use the MixedParamHybridUrlCodingStrategy instead of HybridUrlCodingStrategy to demonstrate the issue. It also includes a fix.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira