Agregar evento onsubmit dinámicamente

[omarMusic]

Holas a todos, tengo una duda cual es la forma correcta de agregar el evento onsubmit a un <form> dinamicamente o sea un equivalente a onsubmit="return verifica(this)", de la forma q a continuación la hago no me funciona correctamente

Html:

<form action="prueba.php" method="post" id="loginForm">
...
</form>

Este es mi código JS

Código Javascript:

window.onload = function() {
    addEvent("submit", "loginForm", function() {return verifica(this)})
}
 
function verifica (form) {
    with(form) {
        if(usuario.value == "") {
            alert("Llene el campo usuario");
            return false;
        }
        if (password.value == "") {
            alert("Llene el campo password");
            return false;
        }
    }
    return true;
}
 
function addEvent(event, elem, func) {
    elem = document.getElementById(elem)
    if (elem.addEventListener) {  // W3C DOM
        elem.addEventListener(event,func,false)
        return true
    } else
        if (elem.attachEvent) // IE DOM
            return elem.attachEvent("on"+event, func)
        else {
            throw 'No es posible añadir evento'
            return false
        }
}
 

Gracias, espero sugerencias
Saludos!

[zerokilled]

buenas,
el evento se registra correctamente. tu problema debe ser la inhabilidad de poder detener el envio cuando los datos no validan bien. esto por la forma en que se registro el evento, addEventListener. cuando se usa la funcion estandar para registrar eventos, la forma de cancelar el evento es un tanto distinto. en lugar de devolver un boolean false debes invocar la funcion preventDefault desde el objeto event que captura el handler. no obstante, en iexplorer -por tener un modelo distinto- debes asignar un boolean a la propiedad returnValue del objeto event.

en fin, dentro de la funcion anonima que has registrado como handler debes escribir arguments[0].preventDefault() -modelo DOM- y arguments[0].returnValue = false -modelo iexplorer-.

[omarMusic]

Muchas gracias hice el siguiente cambio y funciona, espero te hayas referido a esto, talvez hay una mejor manera, me hiciste recuerdo a event.preventDefault(); de jquery

addEvent("submit", "loginForm", function() {if(!verificaLogin(this))arguments[0].preventDefault();arguments[0].returnValue=false;})

Saludos!

[zerokilled]

esta mal. fijate que en el codigo no estas considerando si el navegador es compatible con el modelo DOM o si se trata de iexplorer. en esa porcion estas diciendo algo como si verificaLogin no valido exitosamente, invoca preventDefault. de lo contrario asigna false a returnValue. en realidad lo que debes hacer es si verficaLogin no valida con exitosamente, invocar preventDefault y asignar false a returnValue. de lo contrario, hacer nada. ahora bien, en el proceso de invocar ambas lineas debes tener cuidado puesto que si intentas invocar preventDefault en iexplorer -supongo que desde la version 9 ya funciona-, este te genera error y se detiene la ejecuccion. debes invocar la funcion dentro de una clausula try/catch.

[omarMusic]

Si tienes razón, bueno esta vez lo modifique de ésta manera, ahora si esta correcto

Código Javascript:

addEvent("submit", "loginForm", 
        function() {
            if(!verificaLogin("loginForm")) {
                try {
                    arguments[0].preventDefault()
                } catch (e){
                    arguments[0].returnValue=false
                }
            }
        })
 

Código Javascript:

function verificaLogin (form) {
    with(document.getElementById(form)) {
        if(usuario.value == "") {
            alert("Llene el campo usuario");
            return false;
        }
        if (password.value == "") {
            alert("Llene el campo password");
            return false;
        }
    }
    return true;
}
 

tube q cambiar en la parte donde enviaba el elemento con this y llamar directamente al formulario por su id, ya q en ie no funcionaba, gracias por las correcciones.

Saludos!