homepage Welcome to WebmasterWorld Guest from 54.163.89.8
register, free tools, login, search, pro membership, help, library, announcements, recent posts, open posts,
Become a Pro Member

Home / Forums Index / Code, Content, and Presentation / JavaScript and AJAX
Forum Library, Charter, Moderator: open

JavaScript and AJAX Forum

    
Adding to my email validation
need to add "." check
Revata

5+ Year Member



 
Msg#: 3977302 posted 6:14 pm on Aug 23, 2009 (gmt 0)

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 == '');
}
//-->
</script>

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.

 

whoisgregg

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



 
Msg#: 3977302 posted 2:54 pm on Aug 24, 2009 (gmt 0)

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:

p2=val.indexOf('.');
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.

Revata

5+ Year Member



 
Msg#: 3977302 posted 5:42 pm on Sep 1, 2009 (gmt 0)

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. :-)

Global Options:
 top home search open messages active posts  
 

Home / Forums Index / Code, Content, and Presentation / JavaScript and AJAX
rss feed

All trademarks and copyrights held by respective owners. Member comments are owned by the poster.
Home ¦ Free Tools ¦ Terms of Service ¦ Privacy Policy ¦ Report Problem ¦ About ¦ Library ¦ Newsletter
WebmasterWorld is a Developer Shed Community owned by Jim Boykin.
© Webmaster World 1996-2014 all rights reserved