I'm generally not a fan of having these things auto load or to set instance variables in callbacks in controllers. It make the code much harder to reason about at a glance.