Hola epsilon88, bienvenido a los foros
:
Bueno, no sé que error te dá (deberías indicarnoslo), pero en principio no tienes bien puesto el bloque en valida...
Con el "if" tienes que realizar 2 cosas (mostrar la alerta y cancelar el envío del formulario), así que esas 2 acciones forman un bloque y debes ponerlo entre llaves, sino ejecuta solo la primera acción (la alerta) y siempre (sin condición) se ejecuta la segunda (la cancelación)...
Supongo que en los radion tienes algo así:
<input type=radio value=0 onclick="cambia(this)" name="ra" />
<input type=radio value=1 onclick="cambia(this)" name="ra" />
Un par de recomendaciones:
Como valor es una variable global, anteponle un var:
var valor="0";
Y como los inputs suelen ser cadena de texto, entrecomíllalo:
function valida() {
if (valor == "0") {
alert ("Debe ingresar un Tipo de Documento");
return false;
}
else return true;
}
Espero que te sirva.
Saludos
