1

tengo este form:

<form action="../procedimientos/guardar_familiar.php" onsubmit="return verificarFormulario" method="post">
    <div class="form-group">
        <label>Primer apellido <b>*</b> : </label>
        <input type="text" class="form-control" name="apellido_1" id="apellido_1" placeholder="Introduce el primer apellido">   
    </div>
    <div class="form-group text-center">
        <button id="btn_guardar_familiar" type="submit" class="btn btn-success w-100 py-100">Guardar</button>       
    </div>
</form>


<script>
        function verificarFormulario () {

        var campo_apellido_1 = document.getElementById("apellido_1");

        if (campo_apellido_1.value.trim().length == 0) {
            // toastr.warning('Warning','Hola mund');
            //alert("INVALIDO");
            toastr.warning('El primero apellido no puede estar vacio!','Error');
            return false;
        };

                //alert("VALIDO");
                return true;
        }

</script>

Si no esta completo el campo apellido_1 te manda el toastr bien, pero cuando esta completo no me lo guarda al llamar al fichero php. El fichero.php funciona porque lo he probado con otras cosas, pero ahora que le he metido la validacion con js ya no llo guarda cuando el campo apellido_1 esta relleno.

2
  • 1
    ¿Podrías compartir también el código PHP? Commented el 18 jun. 2019 a las 12:35
  • Tal como te respondieron debes invocar a la funcion llamandola con los respectivos parentesis: "verificarFormulario()".... Commented el 18 jun. 2019 a las 13:00

2 respuestas 2

3

Siempre debería invocarse la función con paréntesis:

<form action="../procedimientos/guardar_familiar.php" onsubmit="return verificarFormulario()" method="post">

Con ese cambio debería funcionar, te recomiendo darle una mirada a este ejemplo que funciona correctamente.

0
1

Compadrito, no se mucho sobre js, pero creo que te faltan los paréntesis en onsubmit="return verificarFormulario()" y colocar dentro de un else el return.

<form action="testok.php" onsubmit="return verificarFormulario()" method="post">
    <div class="form-group">
        <label>Primer apellido <b>*</b> : </label>
        <input type="text" class="form-control" name="apellido_1" id="apellido_1" placeholder="Introduce el primer apellido">   
    </div>
    <div class="form-group text-center">
        <button id="btn_guardar_familiar" type="submit" class="btn btn-success w-100 py-100">Guardar</button>       
    </div>
</form>

<script>
    function verificarFormulario () {

        var campo_apellido_1 = document.getElementById("apellido_1");

        if (campo_apellido_1.value.trim().length == 0) {
            // toastr.warning('Warning','Hola mund');
            //alert("INVALIDO");
            //toastr.warning('El primero apellido no puede estar vacio!','Error');
            return false;

        }
        else {
            //alert("VALIDO");
            return true;
        }              
    }

</script>
1
  • 1
    En teoría el else no hace falta, porque si no entra al if, hará ese segundo return, y si entra hará el primero. Ya sabes que una vez se detona el return se detiene la ejecución del resto de función JS. Commented el 18 jun. 2019 a las 22:28

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.