What (the heck!) is wrong with this?

nathonjones · November 14th, 2012, 06:34 AM

Dim trademail
trademail = Request.Form("trademail")
Response.Redirect("help.asp?reminder=1&tradema il<%=trademail%>")
...is producing the following error:

Microsoft VBScript compilation (0x800A0409)
Unterminated string constant
/reminder.asp, line 94, column 65

I want to add the form variable "trademail" to my querystring when directing back to my help.asp page.

I initially thought that this would work:
Response.Redirect("help.asp?reminder=1&tradema il=<%=Request.Form("trademail")%>")
...but this produces the following error:

Microsoft VBScript compilation (0x800A03EE)
Expected ')'
/reminder.asp, line 92, column 70

...so I tried this:
Response.Redirect("help.asp?reminder=1&tradema il=<%=Request.Form(" & trademail & ")%>")

...but this takes me back to the Unterminated string constant error!

Would appreciate any advice as it's driving me to distraction now. Thanks.
NJ

mat41 · November 14th, 2012, 05:51 PM

You have ASP delimiters inside ASP delimiters. This will work:

Code:

<%
Dim trademail
trademail = trim(Request.Form("trademail"))
Response.Redirect("help.asp?reminder=1&amp;tradema il" & trademail & "") 
%>

note - you should always trim all request objects.

nathonjones · November 15th, 2012, 09:47 AM

Thanks mat41. It should actually read...

<%
Dim trademail
trademail = trim(Request.Form("trademail"))
Response.Redirect("help.asp?reminder=1&tradema il=" & trademail & "")
%>

...but we got the jist.

BTW... we have no idea why a space is being placed between the "a" and "i" in the word trademail. We've double checked what we've submitted, cut and pasted from Notepad to be sure, but there just isn't a space. Go figure.

Anyways, much appreciated. Thank you.
NJ