[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-3116) use a transaction failed event rather than message

[Sylvain Catudal (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBSEAM-3116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12456920#action_12456920 ] 

Sylvain Catudal commented on JBSEAM-3116:
-----------------------------------------

The solution proposed in this JIRA is a lot more to my liking.  What's in JBSEAM-2989 looks like a workaround to me.   

With this solution, it would be easy to remove whatever default behavior comes with seam and put in yours or none at all.  You could log or you could output different messages based upon the context.  It is way more flexible and should be really easy to implement.

Inserting a blank value in the property file so that the message doesn't show should only be a temporary solution while waiting for a real fix.  It adds a warning and I try to keep them at a minimum.  If there are too many warnings, you stop looking at them and you miss the important ones.

> use a transaction failed event rather than message
> --------------------------------------------------
>
>                 Key: JBSEAM-3116
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-3116
>             Project: Seam
>          Issue Type: Feature Request
>          Components: Core
>    Affects Versions: 2.1.0.A1
>            Reporter: Dan Allen
>            Priority: Minor
>             Fix For: 2.1.0.GA
>
>   Original Estimate: 2 hours
>  Remaining Estimate: 2 hours
>
> A FacesMessage should not be added by the SeamPhaseListener when the transaction fails. Instead, it should raise an event that would be observed by a component named FacesTransactionEvents. That observer would then add the FacesMessage to the response. This allows developers to override the behavior or perhaps add a JSF or log message.
> @see FacesSecurityEvents

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira