Shouldn't this be in the if statement?