Maybe you're right and that it's not related, but this really smells like
URL encoding to me... I've had to deal with this problem before.

Did you try changing the url encoding strategy to your own custom one? If
you do and you still get this problem, then I'll believe you. :-)


dml




> -----Original Message-----
> From: Rajiv Jivan [mailto:[EMAIL PROTECTED]
> Sent: 11 March 2008 00:43
> To: users@wicket.apache.org
> Subject: Re: Problem when MIgrating from 1.2 to 1.3 - Parameters
> removed from HomePage URL
>
>
> This doesnt work. The QueryStringUrlCodingStrategy class isn't hit.
> When going to URL
> http://localhost/helloworld?name=John
>
> the code first calls the method
> WicketServlet.doGet(..)
> which delegates the call to WicketFilter.doGet(..)
>
> As pointed in my first post, WicketFilter.doGet checks if the URL is
> the home page and if so, redirects without appending the parameters.
>
>
> On Fri, Mar 7, 2008 at 8:32 PM, David Leangen <[EMAIL PROTECTED]> wrote:
> >
> >  I dunno.
> >
> >  I thought you were talking about mounted urls, whose urls displays are
> >  rendered by a UrlCodingStrategy. If you're using the
> >  QueryStringUrlCodingStrategy, then my guess is that it's relevant.
> >
> >  If you want to know for sure, try applying the patch to your
> local wicket,
> >  or maybe try adding a break and debugging at the lines changed
> by the patch.
> >  If that doesn't work, then I guess you'd better post your
> question again.
> >
> >
> >
> >  Cheers,
> >  Dave
> >
> >
> >
> >
> >  > -----Original Message-----
> >  > From: Rajiv Jivan [mailto:[EMAIL PROTECTED]
> >
> > > Sent: 8 March 2008 00:28
> >  > To: users@wicket.apache.org
> >
> >
> > > Subject: Re: Problem when MIgrating from 1.2 to 1.3 - Parameters
> >  > removed from HomePage URL
> >  >
> >  >
> >  > Are we talking about the same thing? Your patch removes the
> trailing /
> >  > while in my case without the trailing / the parameters aren't
> >  > recognized.
> >  >
> >  > On Thu, Mar 6, 2008 at 6:11 PM, David Leangen
> <[EMAIL PROTECTED]> wrote:
> >  > >
> >  > >  I've already filed an issue and submitted a patch for this:
> >  > >
> >  > >   https://issues.apache.org/jira/browse/WICKET-1385
> >  > >
> >  > >
> >  > >  Cheers,
> >  > >  Dave
> >  > >
> >  > >
> >  > >
> >  > >
> >  > >
> >  > >  > -----Original Message-----
> >  > >  > From: Rajiv Jivan [mailto:[EMAIL PROTECTED]
> >  > >  > Sent: 7 March 2008 05:47
> >  > >  > To: users@wicket.apache.org
> >  > >  > Subject: Problem when MIgrating from 1.2 to 1.3 - Parameters
> >  > >  > removed from HomePage URL
> >  > >  >
> >  > >  >
> >  > >  > We have a site developed using Wicket 1.2. One of the
> use cases we
> >  > >  > have is to pass in a parameter on the home page, and
> based on that we
> >  > >  > perform some actions. e.g
> >  > >  >
> >  > >  > http://localhost/helloworld?name=John
> >  > >  >
> >  > >  > This functionality seem to be broken in 1.3. When passing in
> >  > >  > parameters on the home page URL they are stripped.
> Going through the
> >  > >  > Wicket I have narrowed down to an issue in WicketFilter
> >  > >  >
> >  > >  >         // Special-case for home page - we redirect to add a
> >  > >  > trailing slash.
> >  > >  >         if (relativePath.length() == 0 &&
> >  > >  >
> >  > >  >
> >  >
> !Strings.stripJSessionId(servletRequest.getRequestURI()).endsWith("/"))
> >  > >  >         {
> >  > >  >             final String redirectUrl =
> >  > >  > servletRequest.getRequestURI() + "/";
> >  > >  >
> >  > >  >
> servletResponse.sendRedirect(servletResponse.encodeRedirectURL(red
> >  > >  > irectUrl));
> >  > >  >             return;
> >  > >  >         }
> >  > >  >
> >  > >  > The only workaround I have is to change the URL to
> >  > >  > http://localhost/helloworld/?name=John
> >  > >  > Notice the traling / after helloworld
> >  > >  >
> >  > >  > Am I missing something or is this a bug ?
> >  > >  >
> >  > >  > Thanks,
> >  > >  >
> >  > >  > Rajiv
> >  > >  >
> >  > >  >
> ---------------------------------------------------------------------
> >  > >  > To unsubscribe, e-mail: [EMAIL PROTECTED]
> >  > >  > For additional commands, e-mail: [EMAIL PROTECTED]
> >  > >  >
> >  > >  >
> >  > >
> >  > >
> ---------------------------------------------------------------------
> >  > >  To unsubscribe, e-mail: [EMAIL PROTECTED]
> >  > >  For additional commands, e-mail: [EMAIL PROTECTED]
> >  > >
> >  > >
> >  >
> >  > ---------------------------------------------------------------------
> >  > To unsubscribe, e-mail: [EMAIL PROTECTED]
> >  > For additional commands, e-mail: [EMAIL PROTECTED]
> >  >
> >  >
> >
> >
> >  ---------------------------------------------------------------------
> >  To unsubscribe, e-mail: [EMAIL PROTECTED]
> >  For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to