Discussion :

[guidav]

Bonjour

J'ai une erreur qui arrive sur une de mes pages, cependant je fais appel au même script de la même façon à d'autres endroits.

Le code incriminé est le suivant :

Code :

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

onclick="javascript:	isIe = 1;menuHeight = 22;menuWidth = 168;menuItems = new Array();menuItemNum = 0;addMenuItem('Graphique historique','?&page=etat_graph.php&colonne=&vars=YToxOntzOjY6Il90ZXJtZSI7czo0OiIxODAwIjt9','Images/histo.gif');generateMenu(menuItems);if(isIe) showMenu();

Avec pour la fonction (l'endroit où est l'erreur dans mon cas est en gras)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
function addMenuItem(text, url, img, target)
{
	if(img) {
		menuItems[menuItemNum] = new Array(text, url, img, target);
	}
	else if (text) {
		menuItems[menuItemNum] = new Array(text, url, null, target);
	}
	else {
		menuItems[menuItemNum] = new Array();
	}
	menuItemNum++;
}

Quelqu'un pourrait-il m'aider à identifier mon erreur ?

Merci d'avance

Pour info, cette ligne fonctionne parfaitement :

Code :

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

onclick="javascript:	isIe = 1;menuHeight = 22;menuWidth = 168;menuItems = new Array();menuItemNum = 0;addMenuItem('Graphique historique','?&page=marche_organise_deposit_graph.php&colonne=VAL&vars=YTozOntzOjY6Im1hcmNoZSI7czo1OiJFVVJFWCI7czoxMDoib3Blcl9ncmFwaCI7czozOiJzdW0iO3M6NzoibWF0YWJsZSI7czoxNDoiZGVwb3NpdF9jYWx5b24iO30=','Images/histo.gif');generateMenu(menuItems);if(isIe) showMenu();

[julien.63]

salut
quel est le message d'erreur?

[shaun_the_sheep]

Salut,

Je ne vois rien de spécial dans ton code, peux tu préciser l'environement sur le quel tu travailles ? quel navigateur utilises tu ? et quel message d'erreur récupères tu ?

[guidav]

Ah oui, j'avais oublié... l'erreur est "menuItems est indefini" au niveau du code en gras.
Ce que je ne comprends pas, c'est qu'en général ça fonctionne, et très rarement (sur 2 pages sur 200) pas.
J'utilise IE6 sous win2k, sans possibilité d'utiliser autre chose (et ce script ne fonctionne pas sous FF, il fait appel à des fonctions IE).

[shaun_the_sheep]

Peut etre un problème de cache du navigateur ?

[Matthieu2000]

mets

menuItems = new Array();

dans le header...

[guidav]

Comment ça dans le header ? Je le mets déjà une fois par ligne, pour chaque fois ou j'appelle mon script.

[guidav]

Bonjour

Je déterre ce topic pour ajouter quelques précisions, après avoir été confronté à une situation où je n'ai pas pu contourner le problème :
Le code est toujours le même

Code :

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

onclick="javascript:	isIe = 1;menuHeight = 22;menuWidth = 168;menuItems = new Array();menuItemNum = 0;addMenuItem('Graphique historique','?&page=etat_graph.php&colonne=&vars=YToxOntzOjY6Il90ZXJtZSI7czo0OiIxODAwIjt9','Images/histo.gif');generateMenu(menuItems);if(isIe) showMenu();

Il fonctionne quand je ne me trouve pas dans une balise <form>, mais il ne fonctionne pas quand je me trouve dans un formulaire.
Or pour des raisons techniques, il m'est impossible de déplacer mon code et/ou mes formulaires.

Quelqu'un saurait-il pourquoi ce script ne fonctionne pas dans un formulaire HTML ?

Merci d'avance

Pour info, le code complet du script est en PJ (réutilisable, ne fonctionne que sous IE).

[SpaceFrog]

Code :

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

?&page=

y'a pas un & en trop là ?
le premier paramètre ne doit pas être précédé de "&"

[guidav]

Code :

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

?&page=

y'a pas un & en trop là ?
le premier paramètre ne doit pas être précédé de "&"

En effet, mais c'est bien pratique pour enchaîner des arguments dans un ordre quelconque, et surtout, ça n'a pas d'impact dans mon cas !
Merci quand même, je pensais que c'était permis.

[SpaceFrog]

suffit de mettre le ? en dur et de mettre les paramètres dans un array
et de faire un join '&'