Yep, NumberToDate is the only 'UFL' I use as well. It's important to note that with versions 6 and 7 of Crystal, you had to distribute the NumberToDate UFL (U2LTDATE) to every workstation that needed to run a report that included the NumberToDate function. NumberToDate became a standard Crystal function with version 8 and above, so even this requirement went away.
Celeste, if you edit the subreport in the designer, you can refresh just the subreport - this will tell you if it's even picking up the right data. You can also comment out the subreport criteria, and gradually add the criteria back in (or re-establish links one at a time) to see what's happening. In other words, subreports are typically linked to a field value in the main report, but they don't have to be. If you remove all of the subreport links, then each order should print/display the comments for EVERY order (and every comment type). Then gradually introduce subreport links one-at-a-time, until you get the desired result. The links for open orders should be;
OEORDLIN_SQL.Ord_type = OELINCMT_SQL.Ord_Type
OEORDLIN_SQL.Ord_No = OELINCMT_SQL.Ord_No
OEORDLIN_SQL.Line_No = OELINCMT_SQL.Line_Seq_No
Then, the subreport should also have criteria to look at specific comment types - in most cases OELINCMT_SQL.Cmt_Type = "I"
Peter Shirley