Will do. Thanks for your feedback.
On Wed, Oct 1, 2014 at 9:09 AM, Martin Grigorov <[email protected]> wrote: > Hi, > > Agreed! > Please create a ticket with a patch or PR at GitHub. > Thanks! > > Martin Grigorov > Wicket Training and Consulting > https://twitter.com/mtgrigorov > > On Wed, Oct 1, 2014 at 1:51 AM, Guillaume Smet <[email protected]> > wrote: > >> Hi, >> >> Starting with Wicket 6.16, we have issues redirecting to an absolute >> URL containing a fragment: the # is escaped by Wicket. >> >> Changes in the commit >> >> https://fisheye6.atlassian.com/changelog/wicket-git?cs=e1f29e90ff3a35e117bb3ddc5ee2aefd900c13c8 >> are causing the regression but as far as we can see, the commit is not >> the problem: the issue is that Url.parse then toString encodes the >> anchor. >> >> Before this commit, as we were using an absolute Url, we were in the >> case where the Url weren't escaped. >> >> We think the right fix would be to make org.apache.wicket.request.Url >> anchor aware. >> >> I worked on a patch along these lines for 6.x which is here: >> >> https://github.com/openwide-java/wicket/commit/71a9bc8d8bb1008383e4ad82249c912a10bfd5e2 >> . >> >> I posted a quickstart reproducing the issue here: >> http://share.openwide.fr/share/zone/gsmet/3298658837 . >> >> Thoughts? >> >> -- >> Guillaume >>
