[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-2542) Seam-gen generates false s:link in menu.xhtml (the logout link)

[Pete Muir (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBSEAM-2542?page=comments#action_12397103 ] 
            
Pete Muir commented on JBSEAM-2542:
-----------------------------------

1) The semantics are well defined, but not documented. We should fix this.

2) -1. seam-gen is a simple structure for you to extend.

> Seam-gen generates false s:link in menu.xhtml (the logout link)
> ---------------------------------------------------------------
>
>                 Key: JBSEAM-2542
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-2542
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: Documentation
>            Reporter: Mark Schmatz
>             Fix For: 2.0.1.GA
>
>
> Seam-gen generates the following code in the menu.xhtml page:
> <s:link view="/home.xhtml" action="#{identity.logout}" value="Logout" rendered="#{identity.loggedIn}"/>
> We have here both, the 'view' and the 'action' attributes at the same time. Is this well-defined? I would propose to discard the 'view' attribute and replace the EL in the 'action' attribute with:
> #{authenticator.logout}
> Additionally, in the Authenticator class a logout method should be provided (automatically generated by Seam-gen) which invokes 'identity.logout()'. In pages.xml a respective navigation to the "/home.xhtml" page should be performed.
> The advantage - besides having a proper s:link - is that one can do additional stuff in the logout method before and/or after logging out.

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