Shouldn't this go inside the first part of the if statement?