You're absolutely right. This is where the code should go ideally. Earlier on, I had misunderstood and thought he was trying to set focus from one control to another, which you can not do in a valid event.

>Why not put this in the mytextbox1.VALID instead?
>
>store allt(this.value) to mcard_nbr
>SEEK MCARD_NBR
>IF MCARD_NBR<>Card_Nbr .OR. DELETE()
> DO quickmsg WITH "CUSTOMER'S CARD NO. NOT ON FILE"
> RETURN 0
>ENDIF
>
>
>>For some reason the focus does not get set in the following:
>>store allt(thisform.mytextbox1.value) to mcard_nbr
>> SEEK MCARD_NBR
>> IF MCard_Nbr <> Card_Nbr .OR. DELETE()
>> do quickmsg with "CUSTOMER'S CARD NO. NOT ON FILE"
>> thisform.mytextbox1.setfocus()
>> thisform.refresh
>>
>>Is there a fault here?
>>thanks