I think there's a missing case for email.nil?