2 similar input validations on one form

[bldstr]

Hi, new guy here

I need to have two user input dates validated on a single form when the user clicks submit. At the moment I only hvae it validating one date and I'm not sure how to have it check the other, do I need some sort of a loop?

The form code is:

Code: Select all

<form name="startDateForm" method="post" action="pass.jsp" onSubmit="return ValidateForm()">

<input name="projectStartFromDate" type="text" id="projectStartFromDate">

<input name="projectStartToDate" type="text" id="projectStartToDate">

The Validation function is:

Code: Select all

function ValidateForm(){
    var dt=document.startDateForm.projectStartFromDate
    if(dt.value == '') return true;
    else if(isDate(dt.value)==false){
        dt.focus()
        return false
    }
    return true
 }

The isDate function called upon just defines the number of days in each month etc.

Thanks

EDIT:Link to doc if needed

[benbacardi]

can you not just add a couple of lines to your code to make:

Code: Select all

function ValidateForm(){
    var dt=document.startDateForm.projectStartFromDate
    if(dt.value == '') return true;
    else if(isDate(dt.value)==false){
        dt.focus()
        return false
    }
    var dt2=document.startDateForm.projectStartToDate
    if(dt2.value == '') return true;
    else if(isDate(dt2.value)==false){
        dt2.focus()
        return false
    }
    return true
 }

[bldstr]

I've done some experimenting and that's one of the solutions I came up with; unfortunately although it validates the first date filed ok it still allows an invalid entry through on the second and passes the page.

[Davka]

How about:

Code: Select all

function ValidateForm(){
var dt=document.startDateForm.projectStartFromDate
if(dt.value == '')
 
ValidateDate2();
else if(isDate(dt.value)==false){
dt.focus()
return false
}
function ValidateDate2(){
var dt2=document.startDateForm.projectStartToDate
if(dt2.value == '') return true;
else if(isDate(dt2.value)==false){
dt2.focus()
return false
}
return true
}

The above is sloppy and off the top of my head, but you get the idea.