Can you tell us the field type of [Date1] and [Date2]. If they are in fact date field types then your formula is correct. If they are in fact text fields, I guess the question is why are they are text fields and not actual date fields but assuming they are text fields and you want them to be text fields you would have to test like this.
Trim([Date1]) <>"" or Trim([Date2]) <> ""
If in fact these two fields are taped date fields then post back here but your formula does look correct.
------------------------------
Mark Shnier (YQC)
mark.shnier@gmail.com
------------------------------