Why not throwing an exception here? Instead of silently returning void.
Especially considering the moment we dispatch the event: if user is null then there is a serious issue.