[richfaces-issues] [JBoss JIRA] (RF-11806) UIDataAdaptor breaks with Spring Webflow 2.3 when rendering pages for existing flows

[Lukáš Fryč (Issue Comment Edited) (JIRA)]

    [ https://issues.jboss.org/browse/RF-11806?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12650406#comment-12650406 ] 

Lukáš Fryč edited comment on RF-11806 at 12/13/11 7:39 AM:
-----------------------------------------------------------

I have designed fix for this issue:

It is based on moving registration for

{{UIViewRoot#subscribeForViewEvent(...)}}

from constructor
to the {{PostAddToViewEvent}} time.

Component itself needs to register for events ({{PostAddToViewEvent}} and {{PostRestoreStateEvent}}).
                
      was (Author: lfryc):
    I have designed fixed for this issue:

It is based on moving registration for

{{UIViewRoot#subscribeForViewEvent(...)}}

from constructor
to the {{PostAddToViewEvent}} time.

Component itself needs to register for events ({{PostAddToViewEvent}} and {{PostRestoreStateEvent}}).
                  
> UIDataAdaptor breaks with Spring Webflow 2.3 when rendering pages for existing flows
> ------------------------------------------------------------------------------------
>
>                 Key: RF-11806
>                 URL: https://issues.jboss.org/browse/RF-11806
>             Project: RichFaces
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: component-tables
>    Affects Versions: 4.1.0.CR2
>            Reporter: Marius Bogoevici
>            Assignee: Lukáš Fryč
>            Priority: Critical
>             Fix For: 4.1.1.Final
>
>
> The fix introduced for 4.1.0.CR1 by adding event registration to the constructor:
> {code}
> public UIDataAdaptor() {
>         super();
>         getFacesContext().getViewRoot().subscribeToViewEvent(PreRenderViewEvent.class, this);
>     }
> {code}
> breaks when Spring Webflow restores the view in the middle of the flow with a NPE. E.g. when a data table is constructed, the viewRoot is not yet set on the current thread, which causes the exception above. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira