>I think when you say "methods should never call an event" you mean: "don't call click event from keypress"

What he meant is that you would create a method called FinishSalesNote and call this method from the Click event and from the KeyPress event of the form. This method would return .T. if the sales note could be finished. To solve the issue with the KeyPress event, you have to stop the events:

IF Thisform.FinishSalesNote()
  NODEFAULT
ENDIF