You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Dan Retzlaff (Updated) (JIRA)" <ji...@apache.org> on 2011/11/29 23:57:41 UTC
[jira] [Updated] (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 ]
Dan Retzlaff updated WICKET-3956:
---------------------------------
Attachment: myproject.zip
Martin, the patch does not contain path changes. Maybe my adaptation of the existing unit test confused you? Here is a quickstart that demonstrates the issue simply.
I understand why you recommend upgrading to 1.5, but this is not possible for our product at this time.
> (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: myproject.zip, 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