onchange no funciona

[gspablo]

Tengo este código en una página pero no consigo que funcione, tiene que tener algo mal pero por mas que miro no lo veo

Es una función javascript que selecciona todas las opciones, es activado por un checkbox y puesto el evento onclick, he puesto 2 alerts para ver que llega bien, luego otro en la función, pero es que no llega a la funcion porque no ejecuta ni el primer alert dentro de la función, solo ejecuta el de AA que es el primero y lógico

Código Javascript:

<meta name="docclass" content="Completed" />
<meta name="docrights" content="Copywritten Work" />
<link href="admin-style.css" type="text/css" rel="stylesheet" media="screen" />
<link href="admin-color.css" type="text/css" rel="stylesheet" media="screen" />
<title>FIR-TEX News</title>
 
<script type="text/javascript">
function allornot()
{
    alert("ENTRA")
var value = document.getElementById(checklang).value
    alert(value)
if(value == 'on')
{
  for (i=0;i<document.f1.elements.length;i++)
  {
      if(document.f1.elements[i].type == "checkbox")
         document.f1.elements[i].checked=1 
  }
{
else
{
 for (i=0;i<document.f1.elements.length;i++)
  {
      if(document.f1.elements[i].type == "checkbox")
         document.f1.elements[i].checked=0 
  }
}  
 
}
</script>
</head>
 

Código PHP:

<div align="center">
<span>Select All</span>
<input type="checkbox" name="checklang" onclick="alert('aa'); allornot(); alert('bb');"/>
<select name=check[] multiple=multiple size=count($languages)>";
 

ayuda!!! GRacias

[Adler]

Hola

Prueba así. Fíjate en los cambios

Código Javascript:

function allornot(c)
{
    alert("ENTRA")
var valor = document.getElementById(c).value
    alert(valor)
if(valot == 'on')
{
  for (i=0;i<document.f1.elements.length;i++)
  {
      if(document.f1.elements[i].type == "checkbox")
         document.f1.elements[i].checked=1 
  }
}
else
{
 for (i=0;i<document.f1.elements.length;i++)
  {
      if(document.f1.elements[i].type == "checkbox")
         document.f1.elements[i].checked=0 
  }
}  
 
}
 

Código HTML:

<input type="checkbox" name="checklang" id="checklang" onclick="allornot(this.id);"/>
 

Suerte

[gspablo]

Gracias!

Ya funciona, ahora solo queda que funcione lo que quiero hacer jeje

saludss