Get an Error I don't recognize?

Willy Duitt

11-22-2004, 07:09 PM

And if it didn't error out there it would error out on the next line and the line after that...

Is txtCoName a variable (which is undeclared anywhere in the code you provided) or an input name?

If the latter you need to declare the document.form reference and what attribute of the input you are targetting... I would guess the value....

form.txtCoName.value = form.txtCoName.value;

But then again, in the above context... The entire else conditional is redundant and unneeded....

.....Willy

Willy Duitt

11-22-2004, 07:12 PM

just caught that myownself. what I need to happen, is if the user clicks the checkbox, then all content in the Customer info area, gets copied to the Vehicle Pickup area.

and if they then uncheck that same box, all of that 'copied' data in the Vehicle Pickup area gets removed and the fields are all empty.

I realize now, in looking much closer the problem lies with my ELSE area, in that I'm somehow not setting the Vehicle Pickup items to BLANKS....

???

Jim

In that case: form.txtCoName.value = ''

Puffin the Erb

11-22-2004, 07:15 PM

You could do :

else {
for(var i=10;i<form.length;i++)
{
form.elements[i].value="";
}
}

The first element in the second section of the form can be addressed as index position 10 of the whole form using the elements array and then just loop to the end of the elements array.

JVRudnick

11-22-2004, 07:23 PM

but that syntax gets me an error (REALLY new to javascript !!!)

here's what I'm using now....
=======================================================
<SCRIPT LANGUAGE="JavaScript">


</script>
===============================================

What I want to do is to add that new ELSE to the function, that will
set ALL vars in the Vehicle Pickup area back to BLANKS.

Your " at the end of the line didnt' work, nor did ";

???

Jim

Puffin the Erb

11-22-2004, 07:28 PM

I only posted the else.
Here's the full function.

function ShipToBillPerson(form) {
if (form.copy.checked) {
InitSaveVariables(form);
form.txtVehName.value = form.txtCoName.value;
form.txtVehAddress.value = form.txtCoAddress.value;
form.txtVehCity.value = form.txtCoCity.value;
form.VehState.value = form.CoState.value;
form.txtVehZip.value = form.txtCoZip.value;
form.txtVehContactName.value = form.txtCoContactName.value;
form. txtVehPhone.value = form.txtCoPhone.value;
form.txtVehFax.value = form.txtCoFax.value;
form.txtVehEmail.value = form.txtCoEmail.value;
}
else {
for(var i=10;i<form.length;i++)
{
form.elements[i].value=""; //double quotes x 2
}
}
}

JVRudnick

11-22-2004, 07:34 PM

nicely done. Now, I'm going to try to do the same thing for the last area I also have to offer up to the user if they wish to copy same...

DONE! Thanks fellas....

:-)

Jim

JVRudnick

11-22-2004, 08:06 PM

Lo again...

The script which now works perfectly, has a "side effect" that I just found.

When I fill in the user data, then click the COPY checkbox, it does so quite nicely. But as soon as I UNCLICK the COPY checkbox -- this action "crushes" my SUBMIT and RESET buttons to almost nothing...

???

How can that be? ie the buttons way down the page are being shrunken horizontally by 90%

???

jim

Willy Duitt

11-22-2004, 08:22 PM

Puffin the Erb

11-22-2004, 09:01 PM

Willy's right, you didn't have those form elements (submit and reset) in your original code.
The loop is going to the end of the elements array and setting the values of these buttons to nothing. Buttons are sized according to the text label specified by the value attribute so if the value attribute has no value then the buttons will shrink.
There are various ways to fix this. Either stop the loop at the correct point ( before your buttons) or test for the element type before setting its value.:

function ShipToBillPerson(form) {
if (form.copy.checked) {
InitSaveVariables(form);
form.txtVehName.value = form.txtCoName.value;
form.txtVehAddress.value = form.txtCoAddress.value;
form.txtVehCity.value = form.txtCoCity.value;
form.VehState.value = form.CoState.value;
form.txtVehZip.value = form.txtCoZip.value;
form.txtVehContactName.value = form.txtCoContactName.value;
form. txtVehPhone.value = form.txtCoPhone.value;
form.txtVehFax.value = form.txtCoFax.value;
form.txtVehEmail.value = form.txtCoEmail.value;
}
else {
for(var i=10;i<form.length;i++)
{
if(form.elements[i].type!="submit"&&form.elements[i].type!="reset")
{
form.elements[i].value=""; //double quotes x 2
}
}
}
}