[
https://issues.apache.org/jira/browse/WICKET-4636?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13409442#comment-13409442
]
Bertrand Guay-Paquet commented on WICKET-4636:
----------------------------------------------
Andrea: have you seen issue https://issues.apache.org/jira/browse/WICKET-4637 ?
It contains a set of test pages for different types of redirects in different
situations (page constructor, link handler, form handler).
> Using setResponsePage() before page rendering should skip the rendering of
> the page
> -----------------------------------------------------------------------------------
>
> Key: WICKET-4636
> URL: https://issues.apache.org/jira/browse/WICKET-4636
> Project: Wicket
> Issue Type: Improvement
> Components: wicket
> Affects Versions: 1.5.7
> Reporter: Bertrand Guay-Paquet
> Priority: Minor
> Labels: redirect, requestcycle
> Attachments: WICKET-4636.diff
>
>
> This ticket follows from the email thread
> http://markmail.org/thread/uk5y4ddoopntmkgd
> When setResponsePage() is used, the current page is still rendered. The
> response thus generated is then thrown away completely and a replacement
> request handler is used. I can think of 3 reasons why this should be avoided:
> 1-Performance; it's wasteful to render components to discard them right away
> 2-Needs complete components hierarchy; even if setResponsePage() is used in a
> page constructor (e.g. when redirecting depending on page parameters), since
> the page will be rendered, all of its components or some substitutes must be
> added to the page. Otherwise Wicket will throw a missing component exception
> in dev mode.
> 3-Feedback messages; if the needlessly rendered page contains a feedback
> panel, feedback messages registered in the session will be swallowed by the
> thrown away page rendering.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
