This should be moved into the controller as an instance variable. Otherwise looks good to me. =)