Ravi Knox created WICKET-6192:
---------------------------------
Summary: 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
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)
