Error with if statment

This is a function in inputHandler.cs in the RTS. The problem is in the if(CommandMenu.visible == "0").
What happens is I will rightclick it will show, but then I should be able to right click again and have it disappear. The only to problems I can think of is that immediately as it goes to set it to 0 it sets it to 1 somewhere else (Very unlikely). The second thing I think it could be is that the command menu has it's own inputHandler that makes this code not even run, but I can't find it anywhere.

function GuiRTSTSCtrl::onRightMouseDown(%this, %obj, %x, %y, %z)
{
//echo(%obj.getClassName() @ "right clicked on");
%this.setCommandState("None");
%this.selectionLocked = false;
switch$(%obj.getClassName())
{
case "RTSUnit":
%this.onRightMouseDownUnit(%obj);
case "TerrainBlock":
%this.onRightMouseDownTerrain(%x,%y,%z);
}

if(%this.selectionIncludesTeam == true){ //if units are selected and commandmenu isn't visible make it
if(CommandMenu.visible == "0"){
CommandMenu.position = VectorSub(canvas.getCursorPos(),VectorScale(CommandMenu.extent,1/2)); //command menu moves to cursor position
CommandMenu.visible = "1";
}
else{
CommandMenu.visible = "0";
}
}
}

Please Help!
Thank you for any help you can give!