[POO] Fonction ok sous firefox mais pas sous ie

Version imprimable

Bonjour à tous,

la fonction suivante :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
function goVisualiser( nimTemporaire, dateDebutValidite, typeClient, codePays, typeClientSimple ) {
var formName = "repriseCreationForm";
if (typeClient == "GDO" || typeClient == "GEX") {
document.forms[formName].action = "/<%=ReflexConst.WEBAPP_NAME%>/RepriseCreationPMServlet";
document.forms[formName].action.value = '<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>';
document.forms[formName].nimTemporaire.value = nimTemporaire;	
document.forms[formName].dateDebutValidite.value = dateDebutValidite;		
document.forms[formName].typeClient.value = typeClient;			
document.forms[formName].typeClientSimple.value = typeClientSimple;	
document.forms[formName].submit();
} else {
document.forms[formName].action = "/<%=ReflexConst.WEBAPP_NAME%>/PreparationCreationServlet?action=<%=PreparationCreationServlet.ACTION_CHOIX_PAYS_VALIDER%>";
document.forms[formName].nimTemporaire.value = nimTemporaire;	
document.forms[formName].dateDebutValiditeTemporaire.value = dateDebutValidite;
document.forms[formName].dateDebutValidite.value = dateDebutValidite;		
document.forms[formName].typeClient.value = typeClient;			
document.forms[formName].typeClientSimple.value = typeClientSimple;	
document.forms[formName].codePays.value = codePays;		
document.forms[formName].submit();
}
}

ne pose pas de problème sous firefox 3.0.8 mais j'ai une erreur javascript sous ie 7.
J'ai comme message d'erreur :
"Cet objet ne gère pas cette propriété ou cette méthode".
Quelqu'un aurait-il une idée du pourquoi ?
Merci d'avance.

16/04/2009, 10h18
Bovino

Code:

document.forms[formName].action.value = '<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>';

action.value n'existe pas !!! A moins que tu aies donné un nom action à un de tes éléments de formulaire, ce qui expliquerait le comportement aléatoire selon les navigateurs et te prouverait cette règle simple, les noms de variables, id, class, name ou autres ne DOIVENT PAS être des mots résérvés ou des noms de propriétés !
16/04/2009, 10h21
andry.aime

Citation:

A moins que tu aies donné un nom action à un de tes éléments de formulaire, ce qui expliquerait le comportement aléatoire selon les navigateurs et te prouverait cette règle simple, les noms de variables, id, class, name ou autres ne DOIVENT PAS être des mots résérvés ou des noms de propriétés !

Je pense qu'on peut l'éviter,

Code:

document.forms[formName].elements['action'].value = '<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>';

J'ai modifié comme suite mais cj'ai toujours le même problème :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
function goVisualiser( nimTemporaire, dateDebutValidite, typeClient, codePays, typeClientSimple ) {
	var formName = "repriseCreationForm";
	if (typeClient == "GDO" || typeClient == "GEX") {
		document.forms[formName].action = "/<%=ReflexConst.WEBAPP_NAME%>/RepriseCreationPMServlet?action=<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>";
		document.forms[formName].nimTemporaire.value = nimTemporaire;	
		document.forms[formName].dateDebutValidite.value = dateDebutValidite;		
		document.forms[formName].typeClient.value = typeClient;			
		document.forms[formName].typeClientSimple.value = typeClientSimple;	
		document.forms[formName].submit();
	} else {
		document.forms[formName].action = "/<%=ReflexConst.WEBAPP_NAME%>/PreparationCreationServlet?action=<%=PreparationCreationServlet.ACTION_CHOIX_PAYS_VALIDER%>";
		document.forms[formName].nimTemporaire.value = nimTemporaire;	
		document.forms[formName].dateDebutValiditeTemporaire.value = dateDebutValidite;
		document.forms[formName].dateDebutValidite.value = dateDebutValidite;		
		document.forms[formName].typeClient.value = typeClient;			
		document.forms[formName].typeClientSimple.value = typeClientSimple;	
		document.forms[formName].codePays.value = codePays;		
		document.forms[formName].submit();
	}
}

16/04/2009, 10h30
1r2R1

Citation:

Envoyé par andry.aime

Je pense qu'on peut l'éviter,

Code:

document.forms[formName].elements['action'].value = '<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>';

J'ai essayé aussi mais ça ne fonctionn epas.
L'idée est que suivant l'élément sur lequel je clique, l'action à éxécuter est différente.
16/04/2009, 10h33
1r2R1

Mon form est déclaré comme ça :

Code:

<form name="repriseCreationForm" method="post">

et a des champs hidden.

On peut voir le code JS généré et le Formulaire html.
Essaye d'écrire
Code:

1 2 3 document.forms[formName].elements['nimTemporaire'].value = nimTemporaire; document.forms[formName].elements['dateDebutValidite'].value = dateDebutValidite; ....
C'est plus cross-browser

J'ai fait un truc plus cross-over :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
function goVisualiser( nimTemporaire, dateDebutValidite, typeClient, codePays, typeClientSimple ) {
	var formName = "repriseCreationForm";
	if (typeClient == "GDO" || typeClient == "GEX") {
		document.forms[formName].action = "/<%=ReflexConst.WEBAPP_NAME%>/RepriseCreationPMServlet";
		document.forms[formName].elements['action'].value = "<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>";
		document.forms[formName].elements['nimTemporaire'].value = nimTemporaire;
		document.forms[formName].elements['dateDebutValidite'].value = dateDebutValidite;
		document.forms[formName].elements['typeClient'].value = typeClient;
		document.forms[formName].elements['typeClientSimple'].value = typeClientSimple;
		document.forms[formName].submit();		
	} else {
		document.forms[formName].action = "/<%=ReflexConst.WEBAPP_NAME%>/PreparationCreationServlet";
		document.forms[formName].elements['action'].value = "<%=PreparationCreationServlet.ACTION_CHOIX_PAYS_VALIDER%>";
		document.forms[formName].elements['nimTemporaire'].value = nimTemporaire;
		document.forms[formName].elements['dateDebutValiditeTemporaire'].value = dateDebutValidite;
		document.forms[formName].elements['dateDebutValidite'].value = dateDebutValidite;
		document.forms[formName].elements['typeClient'].value = typeClient;
		document.forms[formName].elements['typeClientSimple'].value = typeClientSimple;
		document.forms[formName].elements['codePays'].value = codePays;
		document.forms[formName].submit();
	}
}

mais ça cross toujours pas over ie :)
comment je fait pour avoir le code js généré ?

16/04/2009, 11h13
andry.aime

Tu charges la page depuis IE et tu affiche le source de celui de IE.
Modifie le nom de l'élément action, c'est là que se pose le problème. Il y a un conflit entre l'attribut 'action' du formulaire et le nom de l'élément du formulaire.

dans la définition du formulaire, normalement on écrit

Code:

1
2
 
<form name="repriseCreationForm" method="post" action="blabla">

c'est ce "blabla" là que je veux "paramétrer" en faisant un truc du genre :

Code:

1
2
 
document.forms[formName].action = '/<%=ReflexConst.WEBAPP_NAME%>/RepriseCreationPMServlet?action=<%=RepriseCreationPMServlet.PAGE_PM_TEMPORAIRE_VISU%>';

Code source de la page :

Code:

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
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
 
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
	<TITLE>Référentiel*Isis*Réseau*Export*(Production)</TITLE>
	<META name="Author" content="reflex Project">
	<BASE href="http://CPN87245.diderot.nmpp.fr:9080/reflex/">
	<LINK rel="stylesheet" HREF="theme/nmppDSI.css" TYPE="text/css">
	<META http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
	<!-- <META http-equiv="Expires" content="0"> -->
 
<base href="http://CPN87245.diderot.nmpp.fr:9080/reflex/">
<script language="javascript" type="text/javascript" src="javascripts/whichRadioSelected.js"></script>
<script language="javascript" type="text/javascript">
 
function goVisualiser( nimTemporaire, dateDebutValidite, typeClient, codePays, typeClientSimple ) {
	var formName = "repriseCreationForm";
	if (typeClient == 'GDO' || typeClient == 'GEX') {
		document.forms[formName].action = '/reflex/RepriseCreationPMServlet?action=2';
		document.forms[formName].elements['nimTemporaire'].value = nimTemporaire;
		document.forms[formName].elements['dateDebutValidite'].value = dateDebutValidite;
		document.forms[formName].elements['typeClient'].value = typeClient;
		document.forms[formName].elements['typeClientSimple'].value = typeClientSimple;
		document.forms[formName].submit();		
	} else {
		document.forms[formName].action = '/reflex/PreparationCreationServlet?action=2';
		document.forms[formName].elements['nimTemporaire'].value = nimTemporaire;
		document.forms[formName].elements['dateDebutValiditeTemporaire'].value = dateDebutValidite;
		document.forms[formName].elements['dateDebutValidite'].value = dateDebutValidite;
		document.forms[formName].elements['typeClient'].value = typeClient;
		document.forms[formName].elements['typeClientSimple'].value = typeClientSimple;
		document.forms[formName].elements['codePays'].value = codePays;
		document.forms[formName].submit();
	}
}
 
function goAssocierDO(codePays, typeClient) {
	var formName = "associerDO";
	document.forms[formName].codePays.value = codePays;
	document.forms[formName].typeClient.value = typeClient;
	document.forms[formName].submit();	
}
 
function goSupprimer(){
	var nLigne = -1;
 
	if ( document.repriseCreationForm.clePointMessagerieTemporaire ) {
		nLigne = whichRadioSelected(document.repriseCreationForm.clePointMessagerieTemporaire);	
	}
 
	if (nLigne >= 0) {
		if ( window.confirm('Confirmez-vous la suppression de ce point messagerie en cours de création ?') ) {				
			document.repriseCreationForm.action.value="4";		
			document.repriseCreationForm.submit();
		}	
  	}
  	else {
  		alert ("Veuillez sélectionner une ligne en cliquant sur un des bouton radio.")
  	}
}
 
</script>
<meta name="GENERATOR" content="IBM WebSphere Studio">
</head>
 
<body style="margin-top: 0; margin-left: 0" bgcolor="#FFFFFF">
 
 
 
 
 
 
 
<script
    language="javascript" src="javascripts/rollover_bandeau_point_messagerie.js" type="text/javascript"></script>
<script language="javascript" src="javascripts/non_disponible.js" type="text/javascript"></script>
<script language="javascript" src="javascripts/gestion_bordereau.js" type="text/javascript"></script>
<table background="./images/commun/fondbandeau.gif" width="100%" style="MARGIN-TOP: 0px" cellpadding="0" cellspacing="0">
    <tr>
        <td width="39"><a href="http://www.nmpp.fr" alt="Site des NMPP" target="_blank"><img src="./images/commun/logo_bandeau.gif" width="39" height="57" border="0"></a></td>
        <td valign="top" width="190"><a href="index.jsp" target="_parent"><img src="./images/commun/b_isis_reseau_export_points_messagerie.gif" width="190" height="57" border="0"></a></td>
        <td><img src="./images/commun/fondbandeau.gif" height="5" width="5"></td>
        <td>
 
        <table cellpadding="0" cellspacing="0" border="0" width="100%">
            <tr>
            		<td width="5%" height="25">&nbsp;</td>
                <td width="25%" height="25" align="center"><a href="/reflex/RechercherServlet" onmouseover="ChangeModeBandeau(1,1)" onmouseout="ChangeModeBandeau(1,0)" onclick="ChangeSelectionBandeau(1)"><img name="m_rechercher" src="./images/m_rechercher_of.gif" border="0" alt="Recherche d'un Point messagerie"></a></td>
 
                <td width="25%" height="25" align="center"><a href="/reflex/PreparationCreationServlet" onmouseover="ChangeModeBandeau(0,1)" onmouseout="ChangeModeBandeau(0,0)" onclick="ChangeSelectionBandeau(0)"><img name="m_creer" src="./images/m_creer_of.gif" border="0" alt="Création d'un Point messagerie"></a></td>
                <td width="35%" height="25" align="center"><a href="/reflex/RepriseCreationPMServlet" onmouseover="ChangeModeBandeau(3,1)" onmouseout="ChangeModeBandeau(3,0)" onclick="ChangeSelectionBandeau(3)"><img name="m_reprise_creation" src="./images/m_reprise_creation_of.gif" border="0" alt="Reprise de la création d'un Point messagerie"></a></td>
 
				<td width="10%" height="25">&nbsp;</td>                   	
            </tr>
            <tr>
                <td colspan="5" height="2" background="./images/commun/ligne_rouge.gif"></td>
            </tr>
            <tr height="6">
                <td width="100%" colspan="5">&nbsp;</td>
            </tr>
        </table>
        </td>
        <td align="right">
        <table cellpadding="0" cellspacing="0" border="0" width="100%">
            <tr>
                <td align="right"><a href="http://outils.nmpp.fr/cerber/servlet/fr.nmpp.cerber.LogoutServlet?redirectUrl=http://sesame.nmpp.fr"><IMG SRC="images/commun/i_deco_jaune_small.gif" border="0" alt="Déconnexion"></a></td>
            </tr>
            <tr>
                <td align="right"><a href="http://sesame.nmpp.fr"><IMG SRC="images/commun/i_retour_sesame_jaune_small.gif" border="0" alt="Retour à la liste des applications Sesame"></a></td>
            </tr>
        </table>
        </td>
    </tr>
</table>
 
 
<table width="100%" border="0" style="margin-top: 0px" cellpadding="0" cellspacing="0">
    <tr>
        <td width="15%" height="538" background="./images/commun/fondbandeau.gif" valign="top"><br>
        <br>
        </td>
        <td width="3%" height="538" valign="top"></td>
        <td width="82%" height="538" valign="top"><br>
        <h2>Liste des Points messagerie en cours de création :</h2>
        <form name="repriseCreationForm" method="post">
        <input type="hidden" name="action" value="">
		<input type="hidden" name="nimTemporaire" value=""> 
		<input type="hidden" name="typeClient" value="">   	       		
		<input type="hidden" name="typeClientSimple" value="">
		<input type="hidden" name="codePays" value="">
		<input type="hidden" name="dateDebutValidite" value="">       
		<input type="hidden" name="dateDebutValiditeTemporaire" value="">
 
        <br>
<table border="0" cellpadding="0" cellspacing="0">
<tr><td align="right">
	<a href="javascript:goSupprimer();"><img src="images/boutons/b_supprimer.gif" border="0"></a>
</td></tr>
<tr><td>        
		<table class="classic">
		<TR>
			<TH>&nbsp;</TH>		
			<TH>NIM temporaire</TH>
			<TH>Type</TH>
			<TH>D&eacute;signation</TH>
			<TH>Pays</TH>
		</TR>
 
 
        <tr class="impair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0003;23/09/2004;PL"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0003', '23/09/2004', 'PEX', '816', 'PL');">	              	
                		RXP0003
                	</a>
                </td>
                <td>PEX</td>
                <td>PEX VANUATU TOTO</td>
                <td >VANUATU</td>
        </tr>
 
        <tr class="pair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0004;16/04/2009;DO"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0004', '16/04/2009', 'GEX', '646', 'DO');">	              	
                		RXP0004
                	</a>
                </td>
                <td>GEX</td>
                <td>frzef</td>
                <td >ABU DHABI</td>
        </tr>
 
 
        <tr class="impair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0001;16/04/2009;DO"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0001', '16/04/2009', 'GEX', '480', 'DO');">	              	
                		RXP0001
                	</a>
                </td>
                <td>GEX</td>
                <td>GEX ESSAI 5 - COLOMBIE</td>
                <td >COLOMBIE</td>
        </tr>
 
        <tr class="pair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0002;16/04/2009;PL"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0002', '16/04/2009', 'PEX', '054', 'PL');">	              	
                		RXP0002
                	</a>
                </td>
                <td>PEX</td>
                <td>PEX TITI</td>
                <td >LETTONIE</td>
        </tr>
 
 
        <tr class="impair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0008;16/04/2009;DO"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0008', '16/04/2009', 'GDO', '004', 'DO');">	              	
                		RXP0008
                	</a>
                </td>
                <td>GDO</td>
                <td>l</td>
                <td >ALLEMAGNE</td>
        </tr>
 
        <tr class="pair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0007;16/04/2009;DO"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0007', '16/04/2009', 'GDO', '004', 'DO');">	              	
                		RXP0007
                	</a>
                </td>
                <td>GDO</td>
                <td>l</td>
                <td >ALLEMAGNE</td>
        </tr>
 
 
        <tr class="impair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0006;16/04/2009;DO"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0006', '16/04/2009', 'GEX', '646', 'DO');">	              	
                		RXP0006
                	</a>
                </td>
                <td>GEX</td>
                <td>l</td>
                <td >ABU DHABI</td>
        </tr>
 
        <tr class="pair">
 
				<td><input type="radio" name="clePointMessagerieTemporaire" value="RXP0005;17/04/2009;PL"></td>
                <td align="center">
  					<a href="javascript:goVisualiser('RXP0005', '17/04/2009', 'PEX', '036', 'PL');">	              	
                		RXP0005
                	</a>
                </td>
                <td>PEX</td>
                <td>l</td>
                <td >SUISSE</td>
        </tr>
 
 
            </table>
</td></tr>
</table>            
            </form>
            <!-- 
            <form name="associerDO" method="post" action="/reflex/PreparationCreationServlet?action=2">
			<input type="hidden" name="codePays" value="">
			<input type="hidden" name="typeClient" value="">			
			<input type="hidden" name="nimTemporaire" value="">
			<input type="hidden" name="dateDebutValiditeStr" value="">
			<input type="hidden" name="typeClientSimple" value="">
		</form>
		 -->
        </td>
    </tr>
</table>
</body>
 
</html>

16/04/2009, 11h49
Bovino

C'est le input qui a pour name action qui pose problème, pas la propriété action du formulaire !
Quand tu écris :

Code:

document.forms[formName].action

D'après toi, le navigateur comprend quoi, l'attribut action du formulaire ou l'élément action du formulaire ?
Oui, tu peux contourner l'accès à l'élément avec la méthode proposée par andry (qui est soit dit en passant la meilleure ;)) mais tu ne pourras pas lever l'ambiguité concernant la propriété action du formulaire !
Donc :

Citation:

cette règle simple, les noms de variables, id, class, name ou autres ne DOIVENT PAS être des mots résérvés ou des noms de propriétés !
16/04/2009, 11h55
1r2R1

Merci merci merci !!
J'avais pas fait gaffe que j'avais un input avec action pour nom !!!
Je l'ai viré et ça marche !!
Le navigateur s'emmellait les pinceaux, forcément.
Maintenant l'action du formulaire est bien modifiée suivant l'élément sur lequel je clique !
Encore merci