abudabit — Mon, 23 Aug 2004 04:07:02 +0000

That was an error on this side of things. Didn't have that error in my script.

Abhishek Reddy — Sun, 22 Aug 2004 10:21:45 +0000

if (!radioCheck(theForm.q[1),"Question 1",true)) return false;'
Notice the incorrect bracket there? Change q[1) to q[1].

abudabit — Sun, 22 Aug 2004 09:00:31 +0000

It's pretty large. But here is a version of the page with only 2 questions (the real one has tons):

&lt;script Language="JavaScript"&gt;

function radioCheck(formField,fieldLabel,required)
{
	var result = false;
	for(var j = 0 ; j < formField.length ; ++j) {
    	if(formField[j].checked) {
	 		result = true;
		}
   	}

	if (!result) {
		result = false;
		alert('Please make a selection for ' + fieldLabel +'.');
	}


	return result;
}


function validateForm(theForm)
{
	if (!radioCheck(theForm.q[1),"Question 1",true)) return false;
	if (!radioCheck(theForm.q[2],"Question 2",true)) return false;

	return true;
}
&lt;/script&gt;

<FORM name=gadgets action='gadgets?gadget=Personality' method='post' onsubmit='return validateForm(this)' id=create>

<B>Question 1......</B><BR>
<INPUT type=radio value=0 name="q[1]"> Not At All
<BR><INPUT type=radio value=1 name="q[1]"> A Bit
<BR><INPUT type=radio value=2 name="q[1]"> Somewhat
<BR><INPUT type=radio value=3 name="q[1]"> True
<BR><INPUT type=radio value=4 name="q[1]"> Very True
<BR><BR>

<B>Question 2.....</B><BR>
<INPUT type=radio value=0 name="q[2]"> Not At All
<BR><INPUT type=radio value=1 name="q[2]"> A Bit
<BR><INPUT type=radio value=2 name="q[2]"> Somewhat
<BR><INPUT type=radio value=3 name="q[2]"> True
<BR><INPUT type=radio value=4 name="q[2]"> Very True
<BR><BR>

<INPUT type=submit name="submit" value="Submit">

</FORM>

Now with normally named radio buttons this works, but with the q[arraypointer] named buttons it doesn't. What it does normally is when you submit the form the ValidateForm function runs the radioCheck function for each button set. The error is occuring with I believe javascript not being able to handle the format of theForm.array[pointer]. Normally you would do theForm.q1 instead of theForm.q[1] and that would work but I need q[1] for php purposes.

Abhishek Reddy — Sun, 22 Aug 2004 07:49:43 +0000

Can you show us the whole page?