[
https://issues.apache.org/jira/browse/WICKET-6192?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15375917#comment-15375917
]
Martin Grigorov commented on WICKET-6192:
-----------------------------------------
The idea is to preserve the named path parameters when creating the urls for
ajax callbacks.
Without this the logic for finding the required named parameters fails and
instead of MountedMapper Wicket uses BookmarkableMapper which has lower
compatibility score and acts as a fallback.
> Remove recreateBookmarkablePagesAfterExpiry ceck in
> AbstractBookmarkableMapper#mapHandler
> -----------------------------------------------------------------------------------------
>
> Key: WICKET-6192
> URL: https://issues.apache.org/jira/browse/WICKET-6192
> Project: Wicket
> Issue Type: Bug
> Reporter: Ravi Knox
> Attachments: WICKET-6192.patch, wicket-sandbox.7z
>
>
> In a nutshell that's what we try to archive:
> Save a cookie on a client browser under the path of the current page.
> The page does have a mandatory named parameter!
> If the Session expires. The SessionExpiredPage will read the cookie and start
> a redirect.
> After resolving Ticket
> [WICKET-6182|https://issues.apache.org/jira/browse/WICKET-6182]:
> What happens now is that by the time the AbstractBookmarkableMapper wants to
> generate the URL of the component, before including the existent parameters,
> it does an getRecreateMountedPagesAfterExpiry() check ([see
> here|https://github.com/apache/wicket/blob/f71a2d33c2ff9dbb669066a3dfc7b58229824b0e/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/AbstractBookmarkableMapper.java#L479]).
> which answers false and the existing parameters are not added to the new
> UrlInfo object.
> That in return will lead to a failure when calling setPlaceholders(copy, url)
> ([see
> here|https://github.com/apache/wicket/blob/f71a2d33c2ff9dbb669066a3dfc7b58229824b0e/wicket-core/src/main/java/org/apache/wicket/core/request/mapper/MountedMapper.java#L206])
> and results in a null url return value, that lets wicket default to the
> generic URL we had before.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
