Updating record in DB HELP

[shafshaa]

Hi,

I am updating a website for my final year project at university, I was wondering if anyone can help.

I am trying to write a script that allows the user to update their email address.
which is also their login. I have written the following script, however, its just adds the new email address as oppose to replacing it. can anyone tell me where i'm going wrong?

<%

Dim strNEW 'New email string

Dim strCONFIRM 'Confirmed email string

Dim strOLD 'old email address string


strOLD = Trim(Request.Form("User_Login") )

strNEW = Trim(Request.Form("New_email") )

strCONFIRM = Trim(Request.Form("Confirm_email") )

%>



<!--#include file="dbOpen.asp"-->

<%

If (strNew <> strCONFIRM) Then

Response.Redirect("my_email_retry.asp")

Else

'qry="select user_id from tblRegUser WHERE user_Email ='" + strNEW + "'"
'set rsChk = Conn.execute(qry)
'IF rsChk.EOF THEN
Conn.BeginTrans
qry="Insert Into tblRegUser(user_Email) VALUES('" & strNEW & "')"
Conn.execute(qry)
Conn.CommitTrans
'END IF

Response.Redirect("disp_login.asp")


%>

<% End IF %>

[Wenzley]

Hi

You need to use the SQL Update command rather than the insert command. You need to know which record to update and you would normally use an ID field (usually a unique number) to update the correct record.

However in your case you can probably get away with using their old email address as the field to update, so your query would be, e.g.

qry="Update tblRegUser Set user_Email =  '" & strNEW & "' Where user_Email = '" & strOLD & '" 

In order for this to work, the old email address that the user types in would have to be the same case - if they type it in once in lower case but then later use upper case, this won't work. You could also get around that by converting their input and the database field to lowercase so that the check would work every time.

Hope that helps.

Dale