myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Kurz (JIRA)" <...@myfaces.apache.org>
Subject [jira] Commented: (MYFACES-2473) Make listeners added by tag handlers transient
Date Fri, 15 Jan 2010 16:36:54 GMT

    [ https://issues.apache.org/jira/browse/MYFACES-2473?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12800747#action_12800747
] 

Michael Kurz commented on MYFACES-2473:
---------------------------------------

And if serialization is the way, EventHandler.Listener should be static to make it work.

> 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.


Mime
View raw message