[ https://issues.apache.org/jira/browse/MYFACES-2473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12800843#action_12800843 ]
Leonardo Uribe commented on MYFACES-2473: ----------------------------------------- If we make it transient, events that needs to be triggered before render phase will not work with partial state saving disabled. Yes it is true if serialization is the way, EventHandler.Listener should be static. I was thinking about make it PartialStateHolder, but first we need to implement UIComponent.EventListenerWrapper saveState and restoreState with delta too. > Make listeners added by tag handlers transient > ---------------------------------------------- > > Key: MYFACES-2473 > URL: https://issues.apache.org/jira/browse/MYFACES-2473 > Project: MyFaces Core > Issue Type: Bug > Components: JSR-314 > Affects Versions: 2.0.0-alpha > Reporter: Michael Kurz > Assignee: Leonardo Uribe > Fix For: 2.0.0-alpha-2 > > Attachments: MYFACES-2473.patch, MYFACES-2473_correct_style.patch > > > I had some troubles with system event listeners added by the following tag > handlers: > org.apache.myfaces.view.facelets.tag.composite.InsertChildrenHandler > org.apache.myfaces.view.facelets.tag.composite.InsertFacetHandler > org.apache.myfaces.view.facelets.tag.jsf.core.EventHandler > As they are no StateHolders, are not serializable and do not have a > no-argument constructor, they cause troubles with state management. The > simplest solution would be to make them serializable but I think it should > also be possible to make them transient as the view is rebuilt anyway on > every request. This would generate no state but I'm not 100% sure if there > are any side effects by making the listeners transient (it works for me and I > can't think of any currently). -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.