Select Box Validation

[Battalion]

I am having issues with making sure users select an option from a select
box... here is my code

function checkit(){
if (document.postdata.phonenumber == "&quot{
alert("Please enter a Phone Number"
return false;}
if (document.postdata.request[document.postdata.request.selectedIndex].value
== "&quot{
alert("Please Select a Request"
return false;}
}




<form name=postdata action=?action=add method=post onsubmit='return
checkit()'>
<input type=text name=phonenumber>
<select name=request>
<option value=1>Send manual</option>
<option value=2>Duplicate Product</option>
<option value=3>Did not order</option>
</select>
<input type=submit>
</form>

can someone help me figure this out. I believe it is because
document.postdata.request is undefined until an option is chosen but I don't
know how to fix that.

 

[MrGreed]

You code has 2 things.

1st:

<form name=postdata action=?action=add method=post onsubmit='return
checkit()'>

Should be:

<form name=postdata action=&quot;place in your url&quot; method=post onsubmit='return
checkit()'>

2nd:

<form name=postdata action=&quot;

http://www.dudeee.com&quot;

method=post onsubmit='return checkit()'>
<input type=text name=phonenumber>
<select name=request>
<option value=&quot;&quot;></option>
<option value=1>Send manual</option>
<option value=2>Duplicate Product</option>
<option value=3>Did not order</option>
</select>
<input type=submit>
</form>

add the bolded code, problem is by default it is returning value 1 to your validate code. To force a choice include an empty selection.

MrGreed

http://www.pastame.com

&quot;did you just say Minkey?, yes that's what I said.&quot;

 

[Battalion]

I would prefer not to add a blank option... I forgot to put the correct select line in, it should be
<select name=request size=5>

 

[Lrnmore]

No selection in a dropdown returns a selected index
of &quot;-1&quot;...

if(document.postdata.request.selectedIndex == -1){
alert(&quot;Don't make me throw another alert&quot; }

2b||!2b