Hi there

I've got a sql statement and then a loop to request the data from the recordset; the only problem is that it only pulls out the same record x the amount of entries there are in the database. For example if I have adam in there as the first entry and then 25 other names after that, it pulls out adam 26 times and doesn't display anyother records. It's looping through the table the amount by the amount of records but only pulling out the top record.

Any help would be greatly appreciated.


Adam

Can I see your code?

----------
---Snib---
----------

<><

Here goes. thanks!


strSQL = "SELECT Country, Beach, SName, STown, DateEntered, Review FROM WaveReviews WHERE Deleted = 0 AND Country = '" & request.Form("Country") & "' AND Beach = '" & request.Form("Beach") & "' ORDER BY DateEntered ASC"

    set rsReview = con.execute(strSQL)

    if rsReview.EOF then %>
    <%
    response.Write("<div align=""center"" class=""td"">")
    response.Write("Sorry no reviews exist for '" & request.Form("Beach") & "'.")
    response.Write("&nbsp;Be the first to submit a review")
    response.Write("<p>")
    response.Write("</div>")

    %>

    <%
    else

    strCountry = rsReview("Country")
    strBeach = rsReview("Beach")
    strName = rsReview("SName")
    strTown = rsReview("STown")
    strDate = rsReview("DateEntered")
    strReview = rsReview("Review")
%>
       <%do while not rsReview.EOF%>



      <tr>
        <td colspan="2" bgcolor="<%=colorx%>" class="td"><div align="left"><%=strReview%></div></td>
      </tr>
      <tr>
        <td colspan="2" bgcolor="<%=colorx%>">
          <div align="right" class="td"><em><%=strName%>,&nbsp;<%=strTown%>,&nb sp;<%=strDate%></em></div>
        </td>
      </tr>
            <%rsReview.movenext()%>

                <%loop%>
      <tr>
<%end if%>

You need to place your strings inside the do while not/loop like so:

<%
  else
  Do While Not rsReview.EOF

  strCountry = rsReview("Country")
  strBeach = rsReview("Beach")
  strName = rsReview("SName")
  strTown = rsReview("STown")
  strDate = rsReview("DateEntered")
  strReview = rsReview("Review")
%>


Chris

of course! sorry being daft