myfaces-dev mailing list archives

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


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

View raw message