Thanks for that - the error has gone..although the table wasn't showing in alternating colours but i managed to sort that out myself by changing one line:
//echo '<tr style="background-color:$color">';
to
echo "<tr style=\"background-color:$color\">";
there is also a semi-colon now showing at the bottom of the page i cannot figure out it's outputting that?
someone also mentioned about not having a do while loop on another forum thread...
....A do/while loop would be incorrect in this situation, since PHP would first execute what's inside the do {} portion; inside the loop, however, you reference $row_Recordset1 which won't be defined until after the first iteration (when then while() condition is evaluated).
That's why database loops are almost always while() loops instead of do/while loops....
so is what i've done a really bad methos - will it dramatically reduce time of results - imapact on server speed etc.?