cambiar onclick al hacer click

[cucuru]

hola! estoy haciendo un botón para añadir/eliminar favoritos, entonces al pulsar la primera vez añade, le cambio la imagen y me gustaría también cambiarle el onclick, para si se vuelve a pulsar elimine:

Código:

 function anadirFav(){
          document.getElementById("fav").src="favorito.png";
          document.getElementById("fav").setAttribute=('onClick','eliminarFav();');
           alert("anadir");
 }
  function eliminarFav(){

         document.getElementById("fav").src="no_favorito.png";
          document.getElementById("fav").setAttribute=('onClick','anadirFav();');
           alert("eliminar");
 }

Código HTML:

<input type="image" id="fav" src="no_favorito.png" onClick = "anadirFav()" /> 

El cambio de imagen funciona perfecto, pero no me cambia la función, ¿alguien podría ayudarme?

Gracias!

[djaevi]

cuando usas set attribute tenes q poner click y sin igual xq setAttribute es una funcion, deberia ser:

setAttribute('Click','funcion()');

Si usas firefox estos errores a veces te ayuda a corregirlos la consola de errores, lo menciono xq es una herramienta que me ayuda mucho a corregir los errores de mis scripts :D

Saludos

[cucuru]

acabo de poner esto:

Código:

          document.getElementById("fav").setAttribute('Click','eliminarFav();');

pero sigue sin funcionar.

¿cómo puedo ver la consola de errores de la que hablas? suena muy interesante!! y yo uso firefox.

Gracicas!

[zerokilled]

buenas...
el atributo está mal escrito. debería ser como incialmente lo tenías, "onclick". setAttribute trabaja directamente con el elemento creando un atributo tal como lo harías en el código fuente. por tanto, "click" no es un atributo válido mientras que "onclick" lo es. aunque, puede ocurrir alguna excepción en algún navegador particular, pero estoy casi seguro que ya se debería a una mal implementación de parte del navegador. el problema inicial era que estabas asignando una expresión al método setAttribute.

Código:

.setAttribute=(...)

por otro lado, no se que tan adecuado es asignar el evento mediante setAttribute. recuerdo que -por ejemplo- iexplorer no entendía este tipo de declaración. desconozco si las versiones actuales ya esta corregido. en lo personal, no utilizaría esa metodología -aunque es funcional- más que nada porque presenta algunas limitaciones respecto al uso de otras características del lenguaje. por ejemplo, no puedes implementar closure. mi propuesta es que utilices las propiedades javascript para declarar el evento. es decir, element.onclick = function_name;.