Can't we hide all of this in a helper? if then else in a view is awful. Bad to test and all