Firefox ignores my javascript PROMPT dialog box..... HELP!

Davidasp · 08-14-2009, 11:30 AM

Hey there guys, i've been having a problem lately and i can't seem to find any solution for that. I know you guys could help me, so here it goes:

I have a javascript promt box (as you can see in the code below) it works perfectly fine with internet explorer however it's NOT working in firefox. I tried everything i can think of, but still its not working!

Please check the code below and give me the solution for firefox. Thank You!

<%@LANGUAGE="VBSCRIPT" CODEPAGE="28592"%>
<!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=iso-8859-2" />
<title>Choose Your Language</title>
<script language="javascript" type="text/javascript">
function checklang()
{
var lang = prompt('Choose A Language:'+'\n' + "(Ex: Armenian, Arabic, English, French, Greek, Russian)",'English');
if (lang==null || lang=="")
{lang=""; return;}

if (lang=="Armenian" || lang=="Arabic" || lang=="English" || lang=="French" || lang=="Greek" || lang=="Russian")
{location.href = "sermon-section.asp?lang="+lang};
else
location.reload();
}
</script>
</head>

<body>

<a href="#" onclick="checklang()">Choose Your Language</a>

</body>
</html>

Amphiluke · 08-14-2009, 11:57 AM

Quote:

Code:

if (lang=="Armenian" || lang=="Arabic" || lang=="English" || lang=="French" || lang=="Greek" || lang=="Russian")
{location.href = "sermon-section.asp?lang="+lang};
else
location.reload();
}

Remove the semicolon...

Davidasp · 08-14-2009, 12:12 PM

I'm feeling sooooo stupid, how couldn't i see that

Anyway thank you man !! Although it was a stupid thing i didn't notice, but you saved me

i also gave you a reputation!

Philip M · 08-14-2009, 04:30 PM

This is a good example of the kind of bug which is likely to arise when braces and semi-colons are not properly positioned, or braces omitted. Far better thus:-

Code:

if (lang=="Armenian" || lang=="Arabic" || lang=="English" || lang=="French" || lang=="Greek" || lang=="Russian") {
location.href = "sermon-section.asp?lang="+lang;
}
else {
location.reload();
}
}