myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Galligan (JIRA)" <>
Subject [jira] Commented: (TOMAHAWK-503) Capture and restore saveState Beans and messages when using redirect navigation rule
Date Wed, 20 Sep 2006 17:19:25 GMT
    [ ] 
Kevin Galligan commented on TOMAHAWK-503:

PPS.  The policy I added, mentioned above, is in the zip file called ''
attached to this bug.  Not sure that that was clear.

As stated in a much earlier comment, you use it by calling into the utility methods...




I wasn't sure what to do with 'redirectContext.saveLocale()', so it calls this if either 'trackMessages'
or 'trackMessagesAndBeans' is called.

> Capture and restore saveState Beans and messages when using redirect navigation rule
> ------------------------------------------------------------------------------------
>                 Key: TOMAHAWK-503
>                 URL:
>             Project: MyFaces Tomahawk
>          Issue Type: Improvement
>    Affects Versions: 1.1.5-SNAPSHOT
>            Reporter: Mario Ivankovits
>         Assigned To: Mario Ivankovits
>             Fix For: 1.1.5-SNAPSHOT
>         Attachments: impl.diff,,, sandbox.patch,
sandbox2.patch, tom.diff
> Ok, this is a try to get what is described in the summary.
> The heart of this patch is the RedirectTracker.
> How it works:
> *) In NavigationHandlerImpl when it comes to the redirect the RedirectTracker will capture
all required (save-state beans/messages) data. The tracker appends a special parameter (_rtid=)
to the redirect url so the saved state can be looked up in an map.
> *) In LifeCycleImpl in restoreView the RedirectTracker will check for the special parameter
and lookup the map using its value. If an entry was found the data will be restored.
> *) In UISaveState every bean rematerialized in restoreState will put it in a request-scoped
> Known limitations:
> *) need to make the number of tracked redirects configureable (back-button) currently
its hard-coded to 20
> *) some, as I didnt tested it very well
> I added this patch as a base to discuss what could be made better, though the goal is
to add this functionality to myFaces.
> I hope some else jump in and we get a vital discussion about this topic.

This message is automatically generated by JIRA.
If you think it was sent incorrectly contact one of the administrators:
For more information on JIRA, see:


View raw message