changer le onclick

**kopax** · 18/04/2010, 16h20

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

<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.

**Bovino** · 18/04/2010, 17h12

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**kopax** · 18/04/2010, 17h33

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**kopax** · 18/04/2010, 19h44

Je cherche toujours une solution à mon problème.

Avec des alerts:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

alert(send_color);

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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