Discussion :

[lodan]

Bonsoir,

J'ai fait un script pour copier un libellé court dans un libellé long comme ceci :

Code :

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

1
2
3
4
function copyLibelCourtLong(libel_court, libel_long) 
{
	document.getElementById(libel_long).value = document.getElementById(libel_court).value;
}

Cette fonction fait partie d'un ensembles de fonction qui toutes fonctionnent.

J'ai comme message lors de la tentative d'exécution le message suivant :

copyLibelCourtLong is not a function

Me suis-je trompé sur mon script de copie ?

Merci

[Auteur]

bonjour,

je ne vois pas d'erreur dans ce bout de code. Tu as dû oublier de fermer une accolade dans la fonction qui la précède ce qui fait que copyLibelCourtLong() est inconnue de ta page.

(je présume que libel_court, libel_long sont des id valides).

[lodan]

Oui les id sont corrects.

Mon code :

Code :

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

1
2
3
4
 
<input type="text" name="libelle_court" id="libelle_court">
<input type="text" name="libelle_long" id="libelle_long">
<input  type="button" onClick="copyLibelCourtLong(this.form.libelle_court,this.form.libelle_long)"

[Auteur]

Si tes id sont justes, je ne vois que ce genre d'erreur donc :
Tu as dû oublier de fermer une accolade dans la fonction qui la précède ce qui fait que copyLibelCourtLong() est inconnue de ta page.

[lodan]

Dans mon code source qui englobe plusieurs scripts, les autres scripts fonctionnent. Seul celui là pose problème.

Je vais creuser.

[E.Bzz]

Bonjour,

Oui les id sont corrects.

peut-être, mais ce que tu passe à ta fonction, ce sont les objets eux-même et non leur id

Code :

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

copyLibelCourtLong(this.form.libelle_court,this.form.libelle_long)

avec une syntaxe qui n'est plus valide, d'ailleurs.

Code :

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

this.form.elements['libelle_court'].id

fonctionnerait peut-être mieux.
Ceci dit, ce n'est visiblement pas cette erreur qui génère le message donné ci-dessus

A+

[Auteur]

Bonjour,
peut-être, mais ce que tu passe à ta fonction, ce sont les objets eux-même et non leur id

même pas... je n'ai même pas vu l'erreur

Code javascript :

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

1
2
 
<input  type="button" onClick="copyLibelCourtLong(this.form.libelle_court,this.form.libelle_long)" >

this désigne l'objet bouton et non les input texte. Il faudrait écrire :

Code javascript :

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

1
2
 
<input  type="button" onClick="copyLibelCourtLong('form.libelle_court', 'form.libelle_long')" >

là l'appel à la fonction, et l'envoi des paramètres est juste.

[E.Bzz]

Code javascript :

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

1
2
 
<input  type="button" onClick="copyLibelCourtLong('form.libelle_court', 'form.libelle_long')" >

là l'appel à la fonction, et l'envoi des paramètres est juste.

Ca va générer

Code :

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

document.getElementById('form.libelle_long').value = document.getElementById('form.libelle_court').value;

Pas sûr que ça soit "plus juste", si ?
Je dirais plutôt

Code :

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

onClick="copyLibelCourtLong('libelle_court', 'libelle_long')"

Mais encore une fois, je pense que c'est une autre erreur qui est à l'origine du message donné ....

A+

[Auteur]

Je dirais plutôt

Code :

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

onClick="copyLibelCourtLong('libelle_court', 'libelle_long')"

ouh là oui, je n'ai pas vu le "form"

Mais encore une fois, je pense que c'est une autre erreur qui est à l'origine du message donné ....

Nous sommes d'accord sur ce point