Please forgive my lack of knowledge.  I have used asp / CDONTS to send the results of a pretty simple form to me.  No fancy formatting required.

It has suddenly begun to send blank emails after displaying an error

error '80040108'
/geta.asp, line 23

this is the line:

ObjMail.Body = "Instructions" & vbtab & instructions & vbcrlf&_

I haven't the slightest idea what can be wrong with the line but someone suggested that I need to convert to CDO.

Is there anyplace that can tell me what to do with my simple code or where to get other code to do this?

I will post the code if appropriate but didn't want to overstep etiquette.

Thanks

Are you by any chance sending e-mails inside a loop and reusing the same CDONTS object?

I think I have see something similar to this. Moving the CreateObject to inside the loop (thereby creating a fresh instance every time) seemed to do the trick.

If that doesn't help, can you post the code?

Cheers,

Imar


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

The form validates perfectly. Here is the FORM description:
<FORM ACTION=geta.asp method="post" name=orderform>

Here is geta.asp. The fields are perfectly correct and I've checked them over and over.

<%
Dim instructions,track,bladder,yourname,address,city,s tate,zip,country,phone,creditcard,expdate,email
instructions = Request.Form("instructions")
track = Request.Form("track")
bladder = Request.Form("bladder")
yourname = Request.Form("yourname")
address = Request.Form("address")
city = Request.Form("city")
state = Request.Form("state")
zip = Request.Form("zip")
country = Request.Form("country")
phone = Request.Form("phone")
creditcard = Request.Form("creditcard")
expdate = Request.Form("expdate")
email = Request.Form("email")
Dim kits
kits = Request.Form("kits")
Dim ObjMail
Set ObjMail = Server.CreateObject("CDONTS.NewMail")
ObjMail.To = "[email protected]"
ObjMail.From = "[email protected]"
ObjMail.Subject = "Dinghy Dogs order"
ObjMail.Body = "Instructions" & vbtab & instructions & vbcrlf&_
"Track" & vbtab & track & vbcrlf&_
"Bladder" & vbtab & bladder & vbcrlf&_
"Name" & vbtab & yourname & vbcrlf&_
"Address" & vbtab & address & vbcrlf&_
"City" & vbtab & city & vbcrlf&_
"State" & vbtab & state & vbcrlf&_
"Zip" & vbtab & zip & vbcrlf&_
"Country" & vbtab & country & vbcrlf&_
"Phone" & vbtab & phone & vbcrlf&_
"Card#" & vbtab & credit & vbcrlf&_
"Exp.Date" & vbtab & expdate & vbcrlf&_
"Email" & vbtab & email & vbcrlf&_
"Ordered" & vbtab & kits & vbcrlf&_
ObjMail.Send
Set ObjMail = Nothing
Response.Write"Thank You For Your Order"
%>


can you see any explanation?

Yes, I can see a problem. You're appending the result of the Send instruction to the Body property:

"Ordered" & vbtab & kits & vbcrlf&_
ObjMail.Send

The underscore is used as a line continuation character. So, in effect this is what this line says:

"Ordered" & vbtab & kits & vbcrlf & ObjMail.Send

Apart from being nonsense, I think you're creating some kind of circular reference. You try to append the result of the Send method to the body of the message you're trying to send ;)

Try this:

"Ordered" & vbTab & kits & vbCrLf
ObjMail.Send()

If you get used to adding spaces to your code as much as possible, you'll spot these kind of errors earlier. It won't make a difference how the code runs, but it is so much easier to look at.

Cheers,

Imar


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

So the underscores should be replaced by spaces and each element placed on a separate line?

No, he just means put spaces between your operators:

"Zip" & vbtab & zip & vbcrlf & _
"Country" & vbtab & country & vbcrlf & _
"Phone" & vbtab & phone & vbcrlf & _

You still need the underscore for code line continuation.

Peter
------------------------------------------------------
Work smarter, not harder.

Well, yes, except for the very last one, right before the line with the Send method.

Cheers

Imar


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

I just wanted to say thanks. I had the same problem and that solved the issue. Stupid me. I had the same problem in ASP.net a while back and it was real simple to solve then. I guess creating appz in an IDE can keep things more in check.

"Instructions" & vbtab & instructions & vbcrlf &_
               "Track" & vbtab & track & vbcrlf&_
                "Bladder" & vbtab & bladder & vbcrlf&_
                "Name" & vbtab & yourname & vbcrlf&_
                "Address" & vbtab & address & vbcrlf&_
                "City" & vbtab & city & vbcrlf&_
                "State" & vbtab & state & vbcrlf&_
                "Zip" & vbtab & zip & vbcrlf&_
                "Country" & vbtab & country & vbcrlf&_
                "Phone" & vbtab & phone & vbcrlf&_
                "Card#" & vbtab & credit & vbcrlf&_
                "Exp.Date" & vbtab & expdate & vbcrlf&_
                "Email" & vbtab & email & vbcrlf&_
                "Ordered" & vbtab & kits & vbcrlf

Just replace this with yours and this will work fine...