What's wrong with this script?

[Brian]

I need a button to change text everytime a person clicks it. It should
switch back and forth from "Request" to "Nevermind." For some reason, this
won't work at all. Does anyone see why it doesn't work?
function changeButton(buttonId)
{
if (document.getElementById)
{
var button=document.getElementById(buttonId);
if (button)
{
if (button.childNodes[0])
{
if (button.childNodes[0].nodeValue=="Request")
{
button.childNodes[0].nodeValue="Nevermind";
}
else
{
button.childNodes[0].nodeValue="Request";
}
}
else if (button.value)
{
if (button.value=="Request")
{
button.value="Nevermind";
}
else
}
button.value="Request";
}
}
else //if (button.innerHTML)
{
button.innerHTML="Nevermind2";
}
}
}
}

[Brian]

I guess this would be it:

else
}

I spent 30 minutes trying to figure out why this script wouldn't work.
GAH!

[Douglas Crockford]

> I need a button to change text everytime a person clicks it. It should

switch back and forth from "Request" to "Nevermind." For some reason, this
won't work at all. Does anyone see why it doesn't work?
function changeButton(buttonId)
{
if (document.getElementById)
{
var button=document.getElementById(buttonId);
if (button)
{
if (button.childNodes[0])
{
if (button.childNodes[0].nodeValue=="Request")
{
button.childNodes[0].nodeValue="Nevermind";
}
else
{
button.childNodes[0].nodeValue="Request";
}
}
else if (button.value)
{
if (button.value=="Request")
{
button.value="Nevermind";
}
else
}
button.value="Request";
}
}
else //if (button.innerHTML)
{
button.innerHTML="Nevermind2";
}
}
}
}

(26) : error at character 9: Expected '{' and instead saw '}'.

JSLINT can be very helpful in spotting errors like this.

http://www.crockford.com/javascript/lint.html