Vladimir,

I forgot to take out the month() calls on the ldStart and ldEnd lines of code. Thanks for the catch.

If the table column is date, and the two textboxes are bound to date inputs then all the month() code could be removed, and if I'm reading Mohammed's intent correctly it'll simplify what he's trying to do. But there isn't always a real clear problem definition in these threads. *s*

>

>table1.dat is Date  &&& may be ;o)
>ldStart is Numeric  &&& month
>and
>ldEnd is Numeric    &&& month
>

>
>
>Two variants:
>1.
>

>ldStart=MONTH(thisform.text2.Value)
>ldEnd=MONTH(thisform.text25.Value)
>m.a=UPPER(ALLTRIM(thisform.text18.value ))
>m.f=THISFORM.TEXT24.VALUE
>
>SELECT * ;
>   FROM table1 ;
>   WHERE BETWEEN( MONTH(table1.dat), ldStart, ldEnd ) AND table1.mac1=m.a AND table1.TYP=m.f ;
>   INTO CURSOR MyCursor

>2.
>

>SELECT * ;
>   FROM table1 ;
>   WHERE BETWEEN(table1.dat, thisform.text2.Value, thisform.text25.Value) AND table1.mac1=m.a AND table1.TYP=m.f ;
>   INTO CURSOR MyCursor

>
>I don't know the idea on Mohhamed.