836 refactor reservations controller

Resolves #836 

Actually, I'm going to go and write tests for this so don't merge until then

<!---
@huboard:{"order":905.0,"milestone_order":882,"custom_state":""}
-->

Yea I figured it out afterwards and meant to update my comment. Do you mind
clarifying what you were asking?