HELP THIS IS MY CODE

Gary_M · October 28th, 2003, 10:47 AM

WHY ISNT HIS WORKING!?!?!?!?!

<html>
<head>
<meta http-equiv="Content-Language" content="en-gb">
<meta http-equiv="Content-Type" content="text/html; charset=windows-1252">
<meta name="GENERATOR" content="Microsoft FrontPage 4.0">
<meta name="ProgId" content="FrontPage.Editor.Document">
<title>Update Version Number</title>
</head>
<body>

<p align="center">Update Version Number</p>
<p align="center">The Filename must be in a standard format eg.
v2.html or v2-1.html</p>
<form "update.asp?action=editsave" method="post">
  <p align="center">ECS
  Version       & nbsp;       &nb sp;       &nbsp ;        F ilename</p>
  <p align="center">
  <input type="text" name="ECSVersionB" size="20">      &nbs p;    
  <input type="text" name="FilenameB" size="20"></p>
  <p align="center">
</form>
    <input type="submit" value="Add" name="Add" language="JavaScript" onclick="self.location.href = 'update.asp?action=list'">

    <%
Select Case Request.QueryString("action")
Case "editsave"
            SQL = "UPDATE ECS SET ECSVersion = '" & Request.Form("ECSVersionB") & "', " & "Filename = '" & Request.Form("FilenameB")'"
            set conn = server.createobject("ADODB.Connection")
            conn.ConnectionTimeout=5
            conn.CommandTimeout=5
            conn.open "ecs"
            set rs = conn.execute(SQL)
            rs.movefirst

            loop
            rs.close
            set rs = nothing
            conn.close
            set conn = nothing

        %>
</body>

</html>

Imar · October 28th, 2003, 11:04 AM

It's hard to tell what problem you encounter, since you didn't define "IS NOT WORKING".....

Anyway, an UPDATE statement does not return a recordset, so there is no point in setting the results of Execute to rs.

Also, when the SQL statement was meant to be a SELECT statement, this is another problem:

Code:

set rs = conn.execute(SQL)
rs.movefirst

A recordset created by calling the Execute method of a Connection object is always a read-only, forward-only recordset and therefore does not support MoveFirst.

Finally, Loop doesn't make much sense all by itself. You'll need a While to be able to do a loop.

HtH,

Imar

---------------------------------------
Imar Spaanjaars
Everyone is unique, except for me.