>>This can be streamlined a bit more also.
>>

>>local cMsgText
>>cMsgText = " "
>>if thisform.text1.value=="123"
>> if Alltrim(thisform.feeuser.value)=="khubaib" or Alltrim(thisform.feeuser.value)=="abid"
>>  feeuser=thisform.feeuser.value
>>  Do Form frsptc
>> Else
>>     cMsgText = "not true for feeuser"
>> Endif
>>else
>>     cMsgText = "not true for text1 value"
>>ENDIF
>>if !empty(cMsgText)
>>   = Messagebox(cMsgText, 32, product)
>>endif
>>thisform.release
>>Return .T.
>>

>
>And even more using INLIST() <g>
wasn't sure how far into shortening the code would start being confusing to Khubaib.
Actually, it seems it would be better to do a CASE within the IF..ENDIF