IMO I think it'd be better to just set the properties on the user directly rather than emitting an event and having the event handler do that for you.  Adds a layer of indirection that seems un-necessary to me.