Discussion :

[GandalfZeGrey]

Bonjour,

J'ai un problème tout bidon qui commence a me taper sur les nerfs :
J'ai une JSP avec du code HTML. La page contenait une floppée de Javascript en en-tete alors, j'ai décidé de l'externaliser dans un fichier.
Et bien lorsque je fais ça, j'ai des fonctionalités qui ne marchent plus...

Quelqu'un aurait il deja vecu ca??

[SpaceFrog]

quel genre de fonctionalités ?
seulement certaines ?
tu externalises comment ?

[GandalfZeGrey]

Pour externaliser, j'ai betement fait un copier-coller de tout le code JS dans un fichier appele JSTest.js et j'ai remplace tout ca par : <script type="text/javascript" src="JStest.js" ></script>
Il n'y a pas de probleme de chemin, j'ai deja verifié grace a firebug notammen et le script est bien importé.

Les fonctionnalites qui foirent sont celles qui vont chercher dynamiquement des propriétés à insérer dans un combobox.

Code :

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

1
2
3
<div class="Div_Content">POI Type name 
<select	name="Vehicle_selectPOI" style="width:100%" onchange="Get_Attributes(this,'Vehicle_selectPoiAttributesName', 'Vehicle_txtFilter')">
		</select></div>

Cette combobox affiche differentes categories de poi et la fonction get_attributes va chercher les attributs du POI selectionne pour les mettre dans une seconde combobox :

Code :

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

<div class="Div_Content" id="div001"><select name="Vehicle_selectPoiAttributesName1" style="width:100px;" onchange="change_filter(this);">

Voila, ce code n'est pas le mien et j'essaye (tant bien que mal) de me l'approprier... Si tu as une idee, tu me sauve la vie !!

Merci pour la reponse en tout cas
A+

[SpaceFrog]

DOnc si tu mets un alert en debut de fichier js il s'affiche bien ...


et on peut voir les fonctions incirminées ?

si il y a des paths dans les fonctions ils doivent être relatifs à l'endroit du script pas de la page

[GandalfZeGrey]

Oui pas de pb : voici la fonction Get_Attributes

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
var Get_Attributes = function(element, result, txt)
{
// disable button search
Change_Search_Button("disabled");
 
var poiType = element;
 
var combo_address1 = document.getElementById(result + "1");
var combo_address2 = document.getElementById(result + "2");
var combo_address3 = document.getElementById(result + "3");
var combo_address4 = document.getElementById(result + "4");
 
// case foodtype
var combo_foodtype = document.getElementById(result + "11");
var combo_foodtype2 = document.getElementById(result + "21");
var combo_foodtype3 = document.getElementById(result + "31");
var combo_foodtype4 = document.getElementById(result + "41");
 
 
// clear all values
 
var txt_address1 = document.getElementById(txt + "1");
var txt_address2 = document.getElementById(txt + "2");
var txt_address3 = document.getElementById(txt + "3");
var txt_address4 = document.getElementById(txt + "4");
 
txt_address1.value = "";
txt_address2.value = "";
txt_address3.value = "";
txt_address4.value = "";
 
// delete all option in the list
combo_address1.length = 0;
combo_address2.length = 0;
combo_address3.length = 0;
combo_address4.length = 0;
 
 
if (combo_foodtype != null)
{
	combo_foodtype.length = 0;
	combo_foodtype.style.visibility = "hidden";
 
	if (txt_address1)
	{
		txt_address1.style.visibility = "inherit";
	}
}
 
if (combo_foodtype2 != null)
{
	combo_foodtype2.length = 0;
	combo_foodtype2.style.visibility = "hidden";
 
	if (txt_address2)
	{
		txt_address2.style.visibility = "inherit";
	}
}
 
if (combo_foodtype3 != null)
{
	combo_foodtype3.length = 0;
	combo_foodtype3.style.visibility = "hidden";
 
	if (txt_address3)
	{
		txt_address3.style.visibility = "inherit";
	}
}
 
if (combo_foodtype4 != null)
{
	combo_foodtype4.length = 0;
	combo_foodtype4.style.visibility = "hidden";
 
	if (txt_address4)
	{
		txt_address4.style.visibility = "inherit";
	}
}
 
// if no filter
if (poiType.value == "")
{
	Change_Search_Button("");
	return ;
}
try{
 
	var vRequest = com.ptvag.webcomponent.common.AJAX.createXMLHttpRequest();
 
	if (!vRequest)
		alert('request null');
 
	//The callback-Handler for the AJAX-request
	var fRequestHandler = function() {
		if (vRequest.readyState == 4) {
			document.documentElement.style.cursor = "default";
			var vJsonResponse = JSON.parse(vRequest.responseText);
			if(vJsonResponse.error){
				alert("Error: "+vJsonResponse.error);
			}else{
				//parse the JSON-object and center-zoom map on coordinates
				for (var i = 0; i < vJsonResponse.length; i++)
				{
					//if (attributsExclud.indexOf(vJsonResponse[i],0) < 0)
					//{
 
						if (vJsonResponse[i].value != "")
						{
							insertValue(combo_address1,getOrderPos(combo_address1,vJsonResponse[i].value),vJsonResponse[i].value,vJsonResponse[i].key);
							insertValue(combo_address2,getOrderPos(combo_address2,vJsonResponse[i].value),vJsonResponse[i].value,vJsonResponse[i].key);
							insertValue(combo_address3,getOrderPos(combo_address3,vJsonResponse[i].value),vJsonResponse[i].value,vJsonResponse[i].key);
							insertValue(combo_address4,getOrderPos(combo_address4,vJsonResponse[i].value),vJsonResponse[i].value,vJsonResponse[i].key);
 
							//combo_address1.options[combo_address1.length] = new Option(vJsonResponse[i].value,vJsonResponse[i].key);
							//combo_address2.options[getOrderPos(combo_address2,vJsonResponse[i].value)] = new Option(vJsonResponse[i].value,vJsonResponse[i].key);
							//combo_address3.options[getOrderPos(combo_address3,vJsonResponse[i].value)] = new Option(vJsonResponse[i].value,vJsonResponse[i].key);
							//combo_address4.options[getOrderPos(combo_address4,vJsonResponse[i].value)] = new Option(vJsonResponse[i].value,vJsonResponse[i].key);
						}
 
						if (vJsonResponse[i].values != "")
						{
							for(var j = 0; j < vJsonResponse[i].values.length; j++)
							{
								combo_foodtype.options[combo_foodtype.length] = new Option(vJsonResponse[i].values[j].value,vJsonResponse[i].values[j].key);
								combo_foodtype2.options[combo_foodtype.length] = new Option(vJsonResponse[i].values[j].value,vJsonResponse[i].values[j].key);
								combo_foodtype3.options[combo_foodtype.length] = new Option(vJsonResponse[i].values[j].value,vJsonResponse[i].values[j].key);
								combo_foodtype4.options[combo_foodtype.length] = new Option(vJsonResponse[i].values[j].value,vJsonResponse[i].values[j].key);
 
							}
						}
					//}
				}
			}
 
			Change_Search_Button("");
		}
	}
 
	alert("POI" + poiType.value);
	//Start the ajax-request
	vRequest.open("POST", "mppoiadmin;jsessionid=<%=session.getId()%>", true); 
	vRequest.onreadystatechange = fRequestHandler;
	vRequest.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
	vRequest.send("PoiType=" + poiType.value + "&lang=" + document.getElementById("Main_lang").value);				
}catch(e){
	alert("Error in ajax call: "+e);
}
}

Si tu y vois quelques chose de suspect...

[SpaceFrog]

le chemin là il est bon ?

Code :

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

vRequest.open("POST", "mppoiadmin;jsessionid=<%=session.getId()%>", true

Code :

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

mppoiadmin;jsessionid=<%=session.getId()%>

ce ne serait pas plutot un ? au lieu d'un ;

et le fichier est dans le même repertoire que le fichier js ?

[GandalfZeGrey]

mppoiadmin est une servlet qui est configurée dans web.xml.
Le chemin est bon a priori.
De plus, j'ai une autre jsp pour un autre mode du site, elle est située au meme endroit dans l'arborescence et fait aussi appel à cette fonction get_attributes avec ce meme chemin.