You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2010/09/21 19:13:34 UTC

[jira] Commented: (WICKET-3065) HomePageMapper ignores request to '/' with query string parameters

    [ https://issues.apache.org/jira/browse/WICKET-3065?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12913086#action_12913086 ] 

Hudson commented on WICKET-3065:
--------------------------------

Integrated in Apache Wicket 1.5.x #347 (See [https://hudson.apache.org/hudson/job/Apache%20Wicket%201.5.x/347/])
    WICKET-3065 HomePageMapper ignores request to '/' with query string parameters

Add unit tests for :
- /
- /?name=value


> HomePageMapper ignores request to '/' with query string parameters
> ------------------------------------------------------------------
>
>                 Key: WICKET-3065
>                 URL: https://issues.apache.org/jira/browse/WICKET-3065
>             Project: Wicket
>          Issue Type: Bug
>          Components: wicket
>    Affects Versions: 1.5-M2.1
>            Reporter: Martin Grigorov
>            Assignee: Martin Grigorov
>             Fix For: 1.5-M3
>
>
> Issue a request to http://host:port/contextpath/?something
> Wicket will log an error message like:
> ERROR - RequestCycle               - Unable to execute request. No suitable RequestHandler found. URL=?something
> I think the reason is in HomePageMapper which maps to the configured home page only if there are no query parameters.
> HomePageMapper.java:
> {code}
> public IRequestHandler mapRequest(Request request)
> 	{
> 		if (request.getUrl().getSegments().size() == 0 &&
> 			request.getUrl().getQueryParameters().size() == 0)
> 		{
> 			return new RenderPageRequestHandler(new PageProvider(getContext().getHomePageClass()));
> 		}
> 		else
> 		{
> 			return null;
> 		}
> 	}
> {code}

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.