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":""}
-->

Definitely in the interest of overall improvement. The reservations controller is practically always in a constant state of refactoring anyway :P