oops bad logic