Welcome to WebmasterWorld Guest from

Forum Moderators: open

Message Too Old, No Replies

Adding to my email validation

need to add "." check



6:14 pm on Aug 23, 2009 (gmt 0)

5+ Year Member

I have this email validation which checks to make sure a "@" sign is present but doesn't check for "."

I just need to add this to the function which I thought would be pretty easy but has proved otherwise. Here's my script:

<script type="text/JavaScript">
function MM_findObj(n, d) { //v4.01
var p,i,x; if(!d) d=document; if((p=n.indexOf("?"))>0&&parent.frames.length) {
d=parent.frames[n.substring(p+1)].document; n=n.substring(0,p);}
if(!(x=d[n])&&d.all) x=d.all[n]; for (i=0;!x&&i<d.forms.length;i++) x=d.forms[i][n];
for(i=0;!x&&d.layers&&i<d.layers.length;i++) x=MM_findObj(n,d.layers[i].document);
if(!x && d.getElementById) x=d.getElementById(n); return x;

function MM_validateForm() { //v4.0
var i,p,q,nm,test,num,min,max,errors='',args=MM_validateForm.arguments;
for (i=0; i<(args.length-2); i+=3) { test=args[i+2]; val=MM_findObj(args[i]);
if (val) { nm=val.name; if ((val=val.value)!="") {
if (test.indexOf('isEmail')!=-1) { p=val.indexOf('@');
if (p<1 p==(val.length-1)) errors+='- Please enter a valid email address.\n';
} else if (test!='R') { num = parseFloat(val);
if (isNaN(val)) errors+='- '+nm+' must contain a number.\n';
if (test.indexOf('inRange') != -1) { p=test.indexOf(':');
min=test.substring(8,p); max=test.substring(p+1);
if (num<min max<num) errors+='- '+nm+' must contain a number between '+min+' and '+max+'.\n';
} } } else if (test.charAt(0) == 'R') errors += '- Please enter your '+nm+'.\n'; }
} if (errors) alert('The following error(s) occurred:\n'+errors);
document.MM_returnValue = (errors == '');

I tried adding another if statement that looked like this...

if (test.indexOf('isEmail')!=-1) { p=val.indexOf('.')

...just after the '@' check, but that didn't seem to work.


2:54 pm on Aug 24, 2009 (gmt 0)

WebmasterWorld Senior Member whoisgregg is a WebmasterWorld Top Contributor of All Time 10+ Year Member

It can be very frustrating working with MM_ functions because Macromedia intentionally makes the code hard to work with. No worries, we should be able to sort this out. :)

Find these two lines of code:

if (test.indexOf('isEmail')!=-1) { p=val.indexOf('@'); 
if (p<1 p==(val.length-1)) errors+='- Please enter a valid email address.\n';

And immediately following, add these new two lines:

if (p2<1 p2==(val.length-1)) errors+='- Please enter a valid email address.\n';

That might not be the exact validation you need, but that gets a period check into the function. If this code doesn't work, just post back and we'll do our best to help. :)

P.S. Make sure you change the broken pipes "¦¦" to regular pipes. This forum software breaks the pipes.


5:42 pm on Sep 1, 2009 (gmt 0)

5+ Year Member

Perfect, thanks gregg! I can see now how the function is layed out, it just spits out of MM so ugly I was having a hard time reading through the functions. After cleaning up line breaks and indentation it made more sense to me. :-)

Featured Threads

Hot Threads This Week

Hot Threads This Month