changer le onclick

Bonjour, alors voilà j'essai de modifié le onClick d'une page.
Ce bout de code fonctionne pour la value, mais pas pour le onClick.
Autant dire que je m'y connais très peu en javascript, et l'erreur dois surement être facile à trouvé pour un développeur javascript.

Code:

---

1 2 3

window.opener.document.forms['shoutbox_form'].elements['couleur'].value = new_color; window.opener.document.forms['shoutbox_form'].elements['couleur'].onclick = "insertTag('[COLOR="+new_color+"]','[/COLOR]','shoutbox');";

---

La première ligne change la value de ce bouton, la deuxieme ne fonctionne pas mais devrais modifié le onClick

Par defaut, le bouton est ainsi :

Code:

---

<input type="button" name="couleur" size="7" maxlength="7" value="#000000" class="case" style="width:70px;" onclick="insertTag('','','shoutbox');" />

---

J'éspère que vous quelqu'un pourra m'aider.

Salut.

Tu fais une confusion (assez fréquente du reste) entre la propriété JavaScript onclick et l'attribut HTML onclick.
Dans ta notation, tu attribues à la propriété onclick une chaîne de caractères, ce qui n'est pas le ce que tu souhaites réellement.
Si tu veux modifier l'attribut HTML, il faut passer par setAttribute(), mais cela n'a pas d'intérêt dans ton cas, cela oblige JavaScript à réinterpréter la valeur de l'attribut puis à l'intégrer dans une fonction anonyme. Le mieux est de définir à l'objet HTML la propriété click (comme tu le fais actuellement) et lui attribuer comme valeur une référence à une fonction.
Ceci dit, si tu fais

Code:

---

elementHTML.onclick = uneFonction(paramètres);

---

tu ne vas pas affecter une référence à la fonction mais le résultat du traitement de cette fonction.
Il faut donc faire :

Code:

---

elementHTML.onclick = uneFonction;

---

sauf que là, tu ne peux pas passer de paramètres...
L'astuce pour passer des paramètres est d'englober l'appel de la fonction dans une référence à une fonction anonyme, dans ton cas :

Code:

---

1 2 3

window.opener.document.forms['shoutbox_form'].elements['couleur'].onclick = function(){ insertTag('[COLOR='+new_color+']','[/COLOR]','shoutbox'); }

---

Merci de ta réponse,

En cherchant sur google, j'avais vu qu'il fallait une méthode anonyme.

J'ai donc essayé (j'ai rajouté un id="couleur" à mon bouton):

Code:

---

1 2	window.opener.document.getElementById("couleur"); couleur.onclick = funtion(){ insertTag('[COLOR='+new_color+']','[/COLOR]','shoutbox'); };

---

Mais ça n'a pas fonctionné.

J'ai également essayé avec le code d'origine que ta corrigé, et malheureusement rien ne se passe.

Je n'ai pas de tools comme firebug pour tester mon js et dans la source je vois pas ci ça comment ça changer.

En tout cas, avant l'execution du script, le onclick fonctionne, après l'execution, il ne fonctionne tout simplement plus.
Je vais essayé de voir les possibilités qu'offre firebug

Edit: avec firebug, je ne trouve pas d'options pour voir l'état de la source après que le script ai modifié le onclick.
Peut être c'est moi qui trouve pas, en tout cas je cherche toujours une solution

Je cherche toujours une solution à mon problème.


Avec des alerts:

Code:

---

alert(send_color);

---

l'alerte de ma fonction me renvois le contenu de la fonction tel quel en entier (new_colorpas modifié)

Code:

---

alert(window.opener.document.forms['shoutbox_form'].elements['couleur'].onclick = function(){insertTag('[COLOR='+new_color+']','[/COLOR]','shoutbox');});

---

L'alerte de la ligne me renvois le contenu de la ligne tel quel (new_colorpas modifié)

Code:

---

alert(new_color);

---

L'alerte me renvoi la couleur bien modifié.

C'est probablement une erreur de syntaxe non?