I am against moving these callbacks to prototype. I agree it will make the constructor more readable but will make the events tougher to follow. Also, we don't gain anything in performance because we would still have to use closures for the event listeners to access ```self```. Therefore I moved these event handing code to a _attach method called in the constructor.

I'm against renaming to eventBus to redisEventBus. PresenceManager receives events that are not directly redis pubsub messages so there is no reason to call it redis.

I've renamed redisWrapper to redisEventBus.

One instance's own messages were already looped back to it. There was some logic for immediately sending status events (online/offline, etc.) which was one of the cause of so much complexity in the code. It's still complex but for other reasons.