GitHub user CedricGatay opened a pull request:
https://github.com/apache/wicket/pull/37
Fix for WICKET-4664
Added tests for RequestUtils.decodeParameters() : in order to be able to
test it easily without the mocking or starting a complex requestCycle,
extracted the static call to getCurrentCharset() as a method parameter. The
extracted method is package visible only and visible for tests only.
When RequestUtils.decodeParameters() is called with a queryString beginning
with a "?" it is automatically stripped and thus is not reflected into
pageParameters.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/CedricGatay/wicket WICKET-4664
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/wicket/pull/37.patch
----
commit 96832b461416db3c95af8d2ae0b597b4d33e5066
Author: Cedric Gatay <ced...@gatay.fr>
Date: 2013-04-29T17:43:34Z
Fix for WICKET-4664
Added tests for RequestUtils.decodeParameters() : in order to be able to
test it easily without the mocking or starting a complex requestCycle,
extracted the static call to getCurrentCharset() as a method parameter. The
extracted method is package visible only and visible for tests only.
When RequestUtils.decodeParameters() is called with a queryString beginning
with a "?" it is automatically stripped and thus is not reflected into
pageParameters.
----
