Javascript guru? Need what I think is easy help

[cstreit]

Trying to run this script on a web page that sends some information to a Google API. The pertinent part is this: (removed script tags so it will post)

var email_addr = document.getElementById("EditBTAddres").elements["email_field"];
document.write (email_addr);
window.renderOptIn = function() {
window.gapi.load('surveyoptin', function() {
window.gapi.surveyoptin.render(
{
"merchant_id": 1234567,
"order_id": "Order ID",
"email": return email_addr,
"delivery_country": "US",
"estimated_delivery_date": "",
"opt_in_style": "OPT_IN_STYLE"
});
});
}

My problem is - the email address is not getting sent. Instead it is sending "return email_addr" in the section hightlighted in red. Any ideas

[stomachmonkey]

"email": return email_addr,

"email": "return email_addr",

enclose return email_address in ""

Also, check the form they are filling in, is the field name / value return email_address or return_email_address

Note inconsistent use of underscore between return and email

EDIT: on a side note, try to be more consistent in your use of underscores and capitalization structure. You've got like 3 different styles for the same element. Pick a style you are comfortable with and stick to it, religously. Makes like a lot easier.

[cstreit]

Originally Posted by stomachmonkey

"email": return email_addr,

"email": "return email_addr",

enclose return email_address in ""

Also, check the form they are filling in, is the field name / value return email_address or return_email_address

Note inconsistent use of underscore between return and email

EDIT: on a side note, try to be more consistent in your use of underscores and capitalization structure. You've got like 3 different styles for the same element. Pick a style you are comfortable with and stick to it, religously. Makes like a lot easier.

Thanks. I'm just stealing bits of code. Most of that is Google's API standard code. I only added the first two lines.

So what I added was: var email_addr = document.getElementById("EditBTAddres").elements["email_field"]; because my form is named "EditBTAddres" (yes with the typo) and the field ID is email_field.

I'll try your suggestion. Thanks!

[HardDrive]

Oh how I hate javascript.

Dunno. I use the inspector in Chrome, throw in a bunch of debugging statements and step through to see whats happening. Not very helpful, I know.

[stomachmonkey]

Originally Posted by cstreit

Thanks. I'm just stealing bits of code.

That's what I suspected.

Actually it looks like your issue is what I said before plus the return.

Do this

"email": "email_addr",

You want that because you are looking for what the form is writing to the array.