is there ever a state where you could have ```@user``` and ```@user.email``` would be neither nil nor not-nil? you could axe this else case, I think.