It really seems to me that this class ought to emit this event on itself rather than on the app.  But looking at other controllers, they don't seem to do that.  So maybe it means that this should be in the lib/ directory instead of the controller directory?  I don't think it is worth making that change.   I guess it is okay to emit this like this.  As long as the app doesn't just handle it by emitting a different event.  Only one event should be necessary.