[
https://issues.apache.org/jira/browse/WICKET-6192?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Martin Grigorov updated WICKET-6192:
------------------------------------
Attachment: WICKET-6192.patch
Here is my suggested patch for this issue.
> 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)
