onClick handler

[FederalProgrammer]

I need to display at most 3 radio buttons depending on the logged-in user (silver members will get 1 radio button, while for a gold member there will be 2 radio buttons displayed and for plat members, there will be 3 radio buttons);

Radio buttons are displayed correctly; My problem is with the onclick function (a javascript function).
My gold and plat members have no problem. But silver members (w/ a single radio button) fail:

for ($i = 0;$i < count($rooms); $i++)
{
$result = mysql_query("SELECT * FROM rooms WHERE id={$rooms[$i]}");
$row = mysql_fetch_object($result);
					
$roomNum = $rooms[$i];
echo "<input type='radio' name='rdiChat' value='$roomNum' onclick = HandleChatroomSelected();>$row->name</a>";
}

following javascript fails (for rdiChat.length = 1)!!

var selectedChat = 'Donot know yet';
for (i = 0; i < document.chat.rdiChat.length; i++)
{
    //this line doesn't get executed for rdiChat.length = 1:
    document.write(document.chat.rdiChat.length);

    if (document.chat.rdiChat[i].checked)
    {
	selectedChat = document.chat.rdiChat[i].value;
    }
}

---------------

http://www.enyotic.com/corporate/

 

[Vragabond]

And since you're having a JavaScript problem, javascript forum would be a much better choice to ask your question in.

 

[FederalProgrammer]

thanx for the tip
I thought may be it has anything to do w/ the way I'm defining my onclick event handler though?

I mean, I am not sure where the code is going wrong. I just know that rdiChat is undefined in the script when the user has access to a single radio button
So I'm not sure where this is originated...

once again, thanx for the tip though

cheers!


---------------

http://www.enyotic.com/corporate/

 

[rekenaar]

I think it is because if there is only one radiobutton it isn't an array.

Try something like this in the javascript:

var selectedChat = 'Donot know yet';

if (rdiChat[0]) //checks if it is an array
{
   for (i = 0; i < document.chat.rdiChat.length; i++)
   {
   
    document.write(document.chat.rdiChat.length);

    if (document.chat.rdiChat[i].checked)
    {
    selectedChat = document.chat.rdiChat[i].value;
    } 
}
else // This is executed if there is only one radiobutton
{
    if (document.chat.rdiChat.checked)
    {
    selectedChat = document.chat.rdiChat.value;
    }
}

I didn't test this but I think it might fix your problem.