EVENT_LOGIN_SUCCESSFUL handled twice

[joeyxxx]

I just upgraded to 2.0.1GA and overrode facesSecurityEvents. addLoginSuccessfulMessage() which used to be in identity so I could suppress the "Welcome, user" message.

From my logs, I can tell authenticator.authenticate is only being called once but addLoginSuccessfulMessage(), the event handler for EVENT_LOGIN_SUCCESSFUL(org.jboss.seam.security.loginSuccessful) is being called twice. What gives?

@Name("org.jboss.seam.security.facesSecurityEvents")
@Scope(ScopeType.APPLICATION)
@Install(precedence = APPLICATION, classDependencies = "javax.faces.context.FacesContext")
@BypassInterceptors
@Startup
public class SecurityEventsHandler extends org.jboss.seam.security.FacesSecurityEvents {

 @Logger Log log;

 @Override @Observer(Identity.EVENT_LOGIN_SUCCESSFUL)
 public void addLoginSuccessfulMessage()
 {
 log.info("\n\n\n>>>>>>>>>>>>>>>>>>>>EVENT_LOGIN_SUCCESSFUL handled<<<<<<<<<<<<<<<<<<<<<<<<\n\n\n");
 }


}

My identity.login ff:

@Override
 public String login() {
 super.login();
 return loginFailed ? null : this.destination;
 }

[shane.bryzak]

The event is only being raised once, however it seems that there is a bug that registers the overridden observer as well as your new observer. I've raised this issue in JIRA:

http://jira.jboss.org/jira/browse/JBSEAM-2571

[joeyxxx]

Thanks Shane.
I'm a bit confused here. Why is my overridden method invoked twice? If the addLogin.. in the super class is registered in addition to my overridden addLogin.., I would expect one invocation in the super class and another in the sub class.
I'm I missing something?

[shane.bryzak]

Because the event bus is calling a component method, and as your class is "the" org.jboss.seam.security.facesSecurityEvents, then it's the one that gets invoked.

[joeyxxx]

Thanks again. Makes sense.

[sebastiendeg]

This issue has been solved !

[joeyxxx]

Thanks. Which revision is it?

[shane.bryzak]

It's fixed in latest SVN.