Shouln't this be in the if?