oops wrong placement of if