Erreur javascript incompréhensible

Version imprimable

Bonjour

J'ai un code (généré par du PHP) qui doit m'afficher un menu.
Le problème est que quand j'appelle mon script dans un <form>, il renvoie une erreur.
Je suis preneur de toutes les solutions, merci d'avance.

Code qui ne marche pas :
erreur : 'menuItems' est indéfini

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
<html>
<script language="JavaScript" type="text/javascript" src="scripts/contextmenu.js"></script>
 
<body >
 
<form name=form action="" method=GET >
 
<a onclick="javascript: isIe = 1;menuHeight = 22;menuWidth = 148;menuItems = new Array();menuItemNum = 0;addMenuItem('Historique','?&page=page.php&colonne=code&vars=123'); generateMenu(menuItems);if(isIe) showMenu();">toto
 
</form>
 
</body>
</html>

Code qui marche :

Code:

1
2
3
4
5
6
7
8
9
10
<html>
<script language="JavaScript" type="text/javascript" src="scripts/contextmenu.js"></script>
 
<body >
 
<a onclick="javascript: isIe = 1;menuHeight = 22;menuWidth = 148;menuItems = new Array();menuItemNum = 0;addMenuItem('Historique','?&page=page.php&colonne=code&vars=123'); generateMenu(menuItems);if(isIe) showMenu();">toto
 
 
</body>
</html>

Help :(

25/01/2007, 10h53
willoi

par souci de clarte, tu pourrais mettre ton code dans une fonction car c est assez difficile a lire.

peut etre il te manque des guillemets a method="GET" name="form"
et donner le nom form est un peu dangereux car peut etre interprete autrement
25/01/2007, 11h09
guidav

Merci pour ta réponse.
Je sais bien que c'est difficilement lisible, mais c'est impossible de mettre ce code dans une fonction (ou alors je ne sais pas le faire simplement), car il peut y avoir plusieurs appels de la fonction addMenuItem selon le nombre d'éléments de mon menu.

J'ai résolu la question en fermant la balise <form> avant d'appeler mes fonctions (il se trouve que c'est possible dans mon cas, mais pas évident que ça soit toujours pareil).

Le fait d'ajouter des guillemets ou de changer le nom du formulaire ne change absolument rien, mais merci de l'idée.
25/01/2007, 11h12
willoi

ok pour le fait de fermer la balise form avant c est sur.

Par contre, en javascript prends l habitude de mettre des guillemets (simples de preference en fait), car sinon tu aura des surprises.