syntax error (missing operator) query expression

[alexz]

valuA = (request.form("toadd"))
If valuA = "" then
SQL = "UPDATE CourseReg SET attended='Active' WHERE ID
IN("&request.form("toadd")&")"
Set RS = MyConn.Execute(SQL)

End If
MyConn.Close
Set RS = Nothing
Set MyConn = Nothing
the error
syntax error (missing operator) query expression ID IN ()

[Alex zurab]

MyConn.Open "media"
If (Request("toadd") = "") then
SQL = "UPDATE CourseReg SET attended='' WHERE ID
IN("&request.form("toadd")&")"
RESPONSE.WRITE SQL
RESPONSE.END
Set RS = MyConn.Execute(SQL)

That what i get
UPDATE CourseReg SET attended='' WHERE ID IN()

if i remove the if statement, i get
UPDATE CourseReg SET attended='' WHERE ID IN(237)

*** Sent via Developersdex [url]http://www.developersdex.com[/url] ***
Don't just participate in USENET...get rewarded for it!

[Aaron Bertrand - MVP]

> If (Request("toadd") = "") then
> SQL = "UPDATE CourseReg SET attended='' WHERE ID
> IN("&request.form("toadd")&")"

Think about this. You're saying, "if this value is blank, put blank into
the IN clause."

[Ray at]

1. Why would you want to execute if toadd IS equal to ""? Wouldn't you
only want to do it if toadd IS NOT equal to ""?

2. You're using request("toadd") and request.form("toadd"). Perhaps you
have a querystring that isn't helping things.

3. Create a variable named sToAdd or something and use sToAdd =
Request.Form("toadd"). Then use the variable instead of constantly calling
the value from the form and/or ruquest collection.

Ray at home

"Alex zurab" <anonymous@devdex.com> wrote in message
news:eDKpo59RDHA.1804@TK2MSFTNGP11.phx.gbl...

> MyConn.Open "media"
> If (Request("toadd") = "") then
> SQL = "UPDATE CourseReg SET attended='' WHERE ID
> IN("&request.form("toadd")&")"
> RESPONSE.WRITE SQL
> RESPONSE.END
> Set RS = MyConn.Execute(SQL)
>
> That what i get
> UPDATE CourseReg SET attended='' WHERE ID IN()
>
> if i remove the if statement, i get
> UPDATE CourseReg SET attended='' WHERE ID IN(237)
>
> *** Sent via Developersdex [url]http://www.developersdex.com[/url] ***
> Don't just participate in USENET...get rewarded for it!

[Alex zurab]

I tried using variable before and it didn't help.
the reason i want to execute if toadd IS equal to ""
is that the checkbox is already checked if there is a value (i.e it's
been update).
if i uncheck and resubmit i want that field to be null again
so a user can fix the record if it was sumbmitted by mistake (not my
idea but i have boses that want certain things).

This is how i did it.
Response.Write("<form action='add.asp' method=Post>")
if rs("attended") = "Active" then
Response.Write ("<input type=checkbox name=toadd value="""& rs("id")&"""
CHECKED>")
else
Response.Write ("<input type=checkbox name=toadd value="""&
rs("id")&""">")
End if


*** Sent via Developersdex [url]http://www.developersdex.com[/url] ***
Don't just participate in USENET...get rewarded for it!