I can't see where this would be the case, shouldn't the body of  this be in the else clause when ok is not true?