Preserved Topic: mr.maX script with multiple forms (Page 1 of 1)

I have been using mr.maX's form validation scripts which work very nicely of course, but I'm now trying to use them on a page with two forms and I'm not quite sure what I need to adjust to accomodate multiple forms on one page. One form just has a select box and submit button. The second form has three text fields, "date", "headline", and "content". Here's what I have tried, but isn't working:

// Written by mr.maX, http://www.max.co.yu/

function testOption(field, msg) {
if (field.selectedIndex == 0) {
alert(msg);
field.focus();
return false;
}
else
return true;
}

function testBlank(field, msg) {
if (field.value == "") {
alert(msg);
field.focus();
field.select();
return false;
} else
return true;
}

function checkDate(field, msg) {
var timeRe = /^([\d]{1,2}\/[\d]{1,2}\/[\d]{2})$/
if (!timeRe.test(field.value)) {
alert(msg);
field.focus();
field.select();
return false;
} else
return true;
}

function testAndSubmit(form) {
return (
testBlank(form.headline, "Headline (must not be blank)") &&
testBlank(form.content, "Article (must not be blank)") &&
checkDate(form.date, "Date (must not be blank and it's format must be MM/DD/YY )") &&
testOption(form.edit_headline, "You must select a headline to edit")
);
}

Thanks for your help!

-Butcher-

Hey butcher. Assuming you are using something like onSubmit=testandSubmit(this) (the this is the important part) to call the function you can simply call a different function depending on which form is being submitted. The this object sends a reference to the specific form so you don't have to do much rewriting. If you are naming your forms (<form name=form1...) you can put a simple if statement in your testandSubmit function.

function testAndSubmit(form) {

if (form.name = "form1") { //Test submit fields

return testOption(form.SubmitFieldName)

} else {   //test  everything else

testBlank(form.headline, "Headline (must not be blank)") &&

testBlank(form.content, "Article (must not be blank)") &&

checkDate(form.date, "Date (must not be blank and it's format must be MM/DD/YY )") &&

testOption(form.edit_headline, "You must select a headline to edit")

)

} else {

}

I hope this helps it may be a little vague.



:[ Computers let you make more mistakes faster than any other invention in human history, with the possible exceptions of handguns and tequila. ]:

You can also write two different testAndSubmit() functions and call appropriate function on each form...

Something like this:

function testAndSubmitForm1(form) {
&nbsp;&nbsp;&nbsp;&nbsp;return ...
}

function testAndSubmitForm2(form) {
&nbsp;&nbsp;&nbsp;&nbsp;return ...
}

<FORM ONSUBMIT="return testAndSubmitForm1(this)">
first form
</FORM>

<FORM ONSUBMIT="return testAndSubmitForm2(this)">
second form
</FORM>




[This message has been edited by mr.maX (edited 01-07-2002).]

Thanks guys!

I really appreciate you both taking the time to help me and share your knowledge.

-Butcher-