On Fri, May 02, 2014 at 09:52:47AM -0700, Brian Sadler wrote:
> Similar login checking code is started to be scattered

It's similar, but not the same: each instance here has a different requirement (is coach vs is admin, for example), a different message ("Please log in with your admin account" vis "Log in"), and a different redirect ("/admin" or "/coaches" or "/assignments").

So there's not much in the way of duplication... yet. If there's two controllers that needed the same check though, then we'd definitely want to move it to a helper or something, but right now each check is pretty unique, if short.