Problemas con 2 eventos submit y onsubmit de validacion en form

[diegomorales]

Buenas necesito ayuda por favor con lo siguiente:

Tengo 2 eventos al hacer submit del form no me hace la validación y el submit al mismo tiempo, muestra la 1ra alerta pero luego ya da el submit. que también maneja otro evento.
Si me pueden dar una opción por favor. Los enlaces de conexion js del head lo tengo en otro html, estoy trabajando con clases.
Aqui está el codigo:

form.html

Código:

<form name ="client" id="client" method="POST" action="index.php" onsubmit="validar_formulario()">
    <input type="hidden" name="id" id="id" value="<?php print $view->client->getId() ?>">
    <div>
        <label>Nombre</label>
        <input type="text" name="nombre" id="nombre" value = "<?php print $view->client->getNombre() ?>">
        <strong style="color:red">(*)</strong>
    </div>
    <div>
        <label>Apellido</label>
        <input type="text" name="apellido" id="apellido"value = "<?php print $view->client->getApellido() ?>">
        <strong style="color:red">(*)</strong>
    </div>
        <div>
        <label>RUC o CI</label>
        <input type="text" name="RUC" id="RUC" value = "<?php print $view->client->getRUC() ?>">
        <strong style="color:red">(*)</strong>
    </div>
        <div>
        <label>Teléfono</label>
        <input type="text" name="telefono" id="telefono" value = "<?php print $view->client->getTel() ?>">
    </div>   
   
    <div class="buttonsBar">
        
        <input id="cancel" type="button" value ="Cancelar" />
        <input id="submit" type="submit" name="submit" value ="Guardar" />
    </div>
</form>

validar.js

Código:

function validar_formulario(){


    if (document.client.nombre.value.length==0){
        alert("Debe ingresar su nombre")
        document.client.nombre.focus()
        return false;
    }
        if (document.client.apellido.value.length==0){
        alert("Debe ingresar su nombre")
        document.client.apellido.focus()
        return false;
    }
        if (document.client.RUC.value.length==0){
        alert("Debe ingresar su nombre")
        document.client.RUC.focus()
        return false;
    }
    return true;
}

funciones.js

Código:

$(document).ready(function(){ 
 $('#client').live('submit',function(){       
        var params={};
              
        params.action='saveClient';
        params.id=$('#id').val();
        params.nombre=$('#nombre').val();
        params.apellido=$('#apellido').val();
        params.fecha=$('#fecha').val();
        params.direccion=$('#direccion').val();
	params.email=$('#email').val();
        params.RUC=$('#RUC').val();
	params.telefono=$('#telefono').val();
        $.post('index.php',params,function(){
            $('#block').hide();
            $('#popupbox').hide();
            $('#content').load('index.php',{action:"refreshGrid"});
        })
        return false;
    })
})

NS={};

[Dradi7]

diegomorales

Primero veo que estas mezclando un framework javascript (jQuery) y luego puro javascript seria bueno que te decidas cual vas a usar.

Primero trata de usar por favor colecciones

document.forms["myform"].elements["myelement"].value

Segundo si ya tienes en el evento onsubmit la parte de validar no seria ahi mismo que realizes el proceso del ajax ya que no vas a recargar la pagina y debes poner siempre return false o return 0.

Código Javascript:

function validar_formulario(){
    if (document.client.nombre.value.length==0){
        alert("Debe ingresar su nombre")
        document.client.nombre.focus()
        return false;
    }
        if (document.client.apellido.value.length==0){
        alert("Debe ingresar su nombre")
        document.client.apellido.focus()
        return false;
    }
        if (document.client.RUC.value.length==0){
        alert("Debe ingresar su nombre")
        document.client.RUC.focus()
        return false;
    }
    var params={};   
    params.action='saveClient';
    params.id=$('#id').val();
    params.nombre=$('#nombre').val();
    params.apellido=$('#apellido').val();
    params.fecha=$('#fecha').val();
    params.direccion=$('#direccion').val();
    params.email=$('#email').val();
    params.RUC=$('#RUC').val();
    params.telefono=$('#telefono').val();
    $.post('index.php',params,function(){
            $('#block').hide();
            $('#popupbox').hide();
            $('#content').load('index.php',{action:"refreshGrid"});
    });
    return false;
}
 

Tercero el evento live ya se encuentra deprecado por favor cambialo por delegate o on en todo caso este punto ya no seria aplicado si realizes lo que te comente en el punto dos.

Saludos

[diegomorales]

Muchas gracias por los consejos. Mis conocimientos en javascript son básicos y los codigos en realidad son con arquitectura MVC que estoy utilizando en el ABM de un script que estoy usando como recurso. Tendría que fijarme mas en esa parte para comprender mejor.

En el caso del código de ayuda, me da el mismo error :( . Ya le dí muchas vueltas pero las mirare mas con el sistema de código que me diste.