Forum Moderators: open

Message Too Old, No Replies

why email validation does not work?

         

lindaonline15

1:55 pm on Nov 12, 2008 (gmt 0)

10+ Year Member



hi all. Im trying to validate my page with javascript, all validations work except for the email address. can anyone tell me whats wrong in here and what I can do to solve it?
thank you

this is the code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Untitled Document</title>

the validation function**************************************************************************************

<script language="javascript" type="text/javascript">

function formValidator(fr)
{
var check = "";
if (fr.elements['FullName'].value.length<1)
{
check += "Please enter user's full name\n\n";
}
if(fr.elements['email'].value.length<1)
{
check += "Please enter user's email address\n\n";
}
if(fr.elements['username'].value.length<1)
{
check += "Please enter user's username\n\n";
}
if(fr.elements['username'].value.length>12)
{
check += "Username can have maximum 12 characters\n\n";
}
if(fr.elements['password'].value.length<6)
{
check += "Please enter user's password with at least 6 characters\n\n";
}
if(fr.elements['AccessLevel'].value.length<1)
{
check += "Please user's level of access\n\n";
}

var emailExp = /^[\w\-\.\+]+\@[a-zA-Z0-9\.\-]+\.[a-zA-z0-9]{2,4}$/;
if(fr.elements['email'].value.match(emailExp)){
return true;
}else{
check += "Please enter a valid email address\n\n";;
email.focus();
return false;
}

if (check != "")
{
alert(check);
return false;
}

}

</script>

</head>

<body>
the form**************************************************************************************

<form action="<?php echo $editFormAction; ?>" method="POST" name="Register" id="Register">
<table width="417" border="0" cellpadding="0" cellspacing="0" class="container-t" align="center">
<!--DWLayoutTable-->
<tr>
<td height="20" colspan="2" valign="top" class="header">Register New Users </td>
</tr>
<tr>
<td width="127" class="labelcell2">Full Name:</td>
<td width="267"><input type="text" name="FullName" maxlength="100" class="textfields" id="FullName"/></td>
</tr>
<tr>
<td class="labelcell2">Email:</td>
<td width="267"><input type="text" name="email" maxlength="100" class="textfields"/></td>
</tr>
<tr>
<td class="labelcell2">Username:</td>
<td width="267"><input type="text" name="username" class="textfields"/></td>
</tr>
<tr>
<td class="labelcell2">Password:</td>
<td width="267"><input type="password" name="password" class="textfields"/></td>
</tr>
<tr>
<td class="labelcell2"> Level of Access:</td>
<td width="267"><select name= "AccessLevel" class="select" id="AccessLevel">
<option value = "" selected="selected">Select </option>
<option value = "1">Admin </option>
<option value = "2">Lecturer </option>
<option value = "3">Student </option>
</select></td>
</tr>
<tr>
<td height="20" colspan="2" valign="top"><input name="submit" type="submit" value="Register the User" class="button" onclick="formValidator(this.form);"/></td>
</tr>
</table>
<input type="hidden" name="MM_insert" value="Register" />
</form>
</body>
</html>

daveVk

10:12 pm on Nov 12, 2008 (gmt 0)

WebmasterWorld Senior Member 10+ Year Member



email.focus();
return false;

Try removing "return false;" so that function continues and displays alert;

Also be consistent and set focus on all on none of the errors.

lindaonline15

3:30 pm on Nov 13, 2008 (gmt 0)

10+ Year Member



thank you very much. I removed both lines and its working now..