Habilitar y desabilitar un campo con Javascript

[pabloe9k]

Hola.
Escribo porque no puedo hacer andar una función JavaScript para habilitar y deshabilitar un campo de texto mediante un checkbox.

La función que estoy utilizando es esta:

Código:

<script>
function habilitaCompras(campoCantidad)
{
	var estadoActual = document.getElementById(campoCantidad)

	if(estadoActual.disabled == 'true')
	{
		estadoActual.disabled=false;
	}
	else
	{
		estadoActual.disabled=true;
	}
</script>

<input type="checkbox" name="producto" onclick="habilitaCompras('cantidad')" />

<br/>

<input type="number" name="cantidad" id="cantidad" min="1" max="99" step="1" size="1" disabled="true" />

No se por qué, pero no logro hacerlo funcionar.
En donde estará mi error?
Como siempre, apreciaré mucho su ayuda.

Gracias.
Saludos.

[diego9j]

Has probado a poner, en una funcion onload, o en el mismo css del campo, el disabled false como predefinido?
Con la funcion toggle visibility pasa algo parecido, el preset no esta bien hecho y hay que hacer esto. Puede ser tu caso


Saludos

[alexg88]

Tienes varios errores.

1. Te faltaba cerrar una llave de la función
2. La condición estadoActual.disabled == "true" es incorrecta, debe ser estadoActual.disabled == true o estadoActual.disabled

Código Javascript:

function habilitaCompras(campoCantidad)
{
    var estadoActual = document.getElementById(campoCantidad);
 
    
    if(estadoActual.disabled)
    {
        estadoActual.disabled= false;
    }
    else
    {
        estadoActual.disabled= true;
    }
}
 

Una versión reducida:

Código Javascript:

function habilitaCompras(campoCantidad)
{
    var estadoActual = document.getElementById(campoCantidad);
 
    estadoActual.disabled = !estadoActual.disabled;
}
 

[pabloe9k]

Iniciado por alexg88

Tienes varios errores.

1. Te faltaba cerrar una llave de la función
2. La condición estadoActual.disabled == "true" es incorrecta, debe ser estadoActual.disabled == true o estadoActual.disabled

Código Javascript:

function habilitaCompras(campoCantidad)
{
    var estadoActual = document.getElementById(campoCantidad);
 
    
    if(estadoActual.disabled)
    {
        estadoActual.disabled= false;
    }
    else
    {
        estadoActual.disabled= true;
    }
}
 

Una versión reducida:

Código Javascript:

function habilitaCompras(campoCantidad)
{
    var estadoActual = document.getElementById(campoCantidad);
 
    estadoActual.disabled = !estadoActual.disabled;
}
 

Tenías razón!! Era la condición que estaba usando (la llave me la comí cuando armé el post).
Muchísimas gracias!!