[WICKET-193] IndexedParamUrlCodingStrategy does not decode PageParameters

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0 branch (discontinued), 1.3.0-beta1
• Fix Version/s: 2.0 branch (discontinued), 1.3.0-beta1
• Environment: all
• Assignee: Jean-Baptiste Quenot
• Reporter: Korbinian Bachl
• Created: 02/Jan/07 14:33
• Updated: 10/Jun/07 16:32
• Resolved: 06/Mar/07 14:09

Description

i noticed that wicket encodes all PageParameters for a page using the IndexedParamUrlCodingStrategy, but never decodes it, so you get the input raw, wich is opposite of the behaviour of BookmarkablePageRequestTargetUrlCodingStrategy where all parameters are encoded and decoded completely - this give great problems in developement as well as the lock to a single strategy in bigger projects as you would need to manually fix it every time you switch.

Furthermore: "We should look at the complete encode/decode story anyway. Now it is done in numerous places some like String.replace("&", "&") and here a decode there a encode. This should all be done for the complete url itself i think in one place (instead per parameter or value)" - JCompagner

Issue Links

• is related to: WICKET-77 missing decoding of encoded PageParameters