chartjes/opencfp — pull request #142 by mdwheele (2014-09-21 02:48:42+00:00)
Title: Bugfix reverts user global changes

After the changes in 443daec, there were a few failing controller tests in commits shortly after. I somehow missed the fact that there were failing tests previously. Most of it stems from adding the Twig environment variable for current authenticated user and removing that reference in many places where template data is specified.

In places, the `user` being passed as template data was not, in fact, the Sentry user. It was a reference to the Spot mapper for user. This probably affects a few other places, but I haven't checked. I'm instead going to go ahead and focus on tests covering the rest of controller methods and will likely catch it then. I'll also try to improve consistency as I move along because I can totally see how a mistake like this could happen.