fixed end of if