Discussion :

[franklin626]

Bonjour,

j'ai une erreur que je ne parviens pas à faire disparaître sous Firebug dans le code suivant :

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
 
function f_report(a_report, report_view) {
28 ltxt = top.TreeFRM.getSelectedDataElementIDs();
29
30 if (ltxt.length > 0) {
31 var trimmedLtxt = ltxt.substr(0, ltxt.length - 1);
32 if (trimmedLtxt.split(":").length > 1) {
33 alert("Please select only one installation.");
34 return;
35 }
36 } else {
37 alert("Please select one installation.");
38 return;
39 }
40
41 var lform = top.ReportInputFRM.document.getElementById("ReportPreview"); // form for year/month etc.
42 var sArgs = "";
43 if (lform.ReportYear)
44 sArgs += "&ReportYear=" + lform.ReportYear.value;
45 if (lform.ReportMonth)
46 sArgs += "&ReportMonth=" + lform.ReportMonth.value;
47 if (lform.ReportHalfYear)
48 sArgs += "&ReportHalfYear=" + lform.ReportHalfYear.value;
49 if (lform.ReportQuarter)
50 sArgs += "&ReportQuarter=" + lform.ReportQuarter.value;
51 if (lform.Persons)
52 sArgs += "&Persons=" + lform.Persons.value;
53 if (lform.WastePort)
54 sArgs += "&WastePort=" + lform.WastePort.value;
55 if(lform.ExportPort)
56 sArgs += "&ExportPort=" + lform.ExportPort.value;
57 if (lform.optionNames) {
58 var ta = lform.document.getElementsByName("optionNames");
59 for ( var i = 0; i < ta.length; ++i) {
60 var theName = ta[i].value;
61 var theValue = lform.document.getElementsByName(theName);
62 if (theName == "OperatingHours") {
63 var is_OperatingHouers = true;
64 }
65 if (theName == "operatenum") {
66 var operatenum_array_pos = i;
67 }
68
69 if (theValue) {
70 for ( var j = 0; j < theValue.length; ++j) {
71 sArgs += "&" + ta[i].value + "=" + theValue[j].value;
72 }
73 } else
74 alert(ta[i].value + " not defined value");
75
76 }
77 // This IF looks for OperatingHours Variable and sends all Hour fields
78 // over to the java code
79 // If the names of the OperatingHours Input Text box are changed this
80 // code also needs changing
81
82 if (is_OperatingHouers) {
83 var turbineNrs = lform.document
84 .getElementsByName(ta[operatenum_array_pos].value)[0].value
85 .split(",");
86
87 for ( var i = 0; i < (turbineNrs.length - 1); i++) {
88 sArgs += "&"
89 + turbineNrs[i]
90 + "="
91 + lform.document.getElementsByName(turbineNrs[i])[0].value;
92 }
93 }
94 }
95 if (lform.eemsuser)
96 sArgs += "&eemsuser=" + lform.eemsuser.value;
97 if (lform.pswd)
98 sArgs += "&pswd=" + lform.pswd.value;
99 if (lform.permitnumber)
100 sArgs += "&PermitNumber=" + lform.permitnumber.value;
101
102 if (lform.submitTest && lform.submitTest.checked)
103 sArgs += "&submitTest=" + "true";
104 else
105 sArgs += "&submitTest=" + "false";
106
107 rframe = top.ReportFRM;
108 rframe.location.href = servletPath
109 + "?method=ReportEEMSRefresh&ReportName=" + a_report + sArgs
110 + "&ReportView=" + report_view + "&InstIDs=" + ltxt
111}

L'erreur apparaît à la ligne 58. Firebug m'indique que le champ .document est inexistant. Cependant, cela passe très bien sous IE. Chaque changement que j'ai tenté fait planter Firebug et le debugger IE.

Manque-t-il un champ id quelque part ? Ou y a-t-il un problème car "lform" est une variable ?

je vous remercie d'avance pour la lecture de ce post !

[SpaceFrog]

Code :

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

var lform = top.ReportInputFRM.document.getElementById("ReportPreview")

lfrom est donc un objet html ...

Code :

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

var ta = lform.document.getElementsByName("optionNames");

document est une propriété de window...


essaye

Code :

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

var ta = lform.getElementsByName("optionNames");

[franklin626]

Oui, déjà tenté, et Firebug me répondait :

-> lform.getElementsByName is not a function

IE la même chose :

-> not a property or method

[franklin626]

je précise le problème que j'ai en premier lieu, Firebug indique que .document est "undefined". Je suppose que la propriété est donc bien valide car IE arrive à y accéder. Elle semble juste "nulle" sous Firefox.

lform est un formulaire, du coup .document est bien valide non ?

[Bovino]

lfrom est donc un objet html ...

J'ai plutôt l'impression que lfrom est un formulaire dont un élément a pour name 'document'

[Bovino]

.document est bien valide non ?

Bien sûr que non : document est un objet natif de JavaScript, nommer des éléments de formulaire comme ça relève du suicide programmatique

[franklin626]

j'ai vérifié mon Html, je n'ai aucun élément du formulaire nommé 'document'.
Du coup, pourquoi IE gère-t-il .document dans ce cas ?

[Bovino]

Là, ce serait bon de nous montrer le code HTML correspondant !

[franklin626]

Voici donc l'HTML :

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
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
 
<!--
<?xml version="1.0" encoding="UTF-8"?>
-->
<!--
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
-->
 
<html>
 <head>
  <title>Add document</title>
  <link type="text/css" rel="stylesheet" href="accounter/calendar.css"/>
<link rel="stylesheet" href="accounter/Nems_ReportDef.css" type="text/css"/>
<style>
td {
  font-size: 12px;
  font-family:  Arial, Helvetica, sans-serif;
  background: transparent;
  white-space: nowrap;
}
 
</style>
 
<script type="text/javascript" src="accounter/jquery/jquery-1.4.2.min.js"></script>
 
<script type="text/javascript" src="accounter/Nems_accounter.js"></script>
 
 
<script src="accounter/Report_Eems.js"></script>
<SCRIPT TYPE="text/javascript" SRC="accounter/grid/taconite-client.js"></SCRIPT> 
<SCRIPT TYPE="text/javascript" SRC="accounter/grid/taconite-parser.js"></SCRIPT> 
 
<script type="text/javascript"><!--
 
var isRunning = 1;
var servletPath = '/NemsAccounter/nems.jhtml';
function f_refresh() {
	f_report('EEAromatic',0); 
}
 
$(document).ready(function () {
    f_set_info_txt( "<BR><H4>OPPC Bi-Annual for <span id='infofacilityname'> Testfacility A</span> </H4>");
   f_set_tree_ok();
});
 
 
function f_ajax_load_yearhours() {
		  if (('EEAromatic'=="EEAtmosDrill") || ('EEAromatic'=="EEDrillFluids") || ('EEAromatic'=='EEWasteNorm')){
			var ajaxRequest = new AjaxRequest('nems.jhtml?method=axYearHoursEEAromatic'); //Create AjaxRequest object
			ajaxRequest.setUsePOST();
			ajaxRequest.addNameValuePair('ReportName','EEAromatic');
	        ajaxRequest.addNameValuePair('ReportYear',document.getElementById("ReportYear").value);
			ajaxRequest.addNameValuePair('InstIDs', top.TreeFRM.getSelectedDataElementIDs());
	        ajaxRequest.sendRequest();  //Send the request
	    }	  
	}
 
 
function is_dates_set(action,repname,x) {
 var lform = top.ReportInputFRM.document.getElementById("ReportPreview");
 // trenger if optionNames 
 if(lform.optionNames) {
 	var ta = lform.document.getElementsByName("optionNames");
		if((ta[0].value =="StartDate" )&&(ta[1].value == "EndDate")){
			if((lform.StartDate.value != "")&&(lform.EndDate.value != "")){	
 
				if(action == "Overview"){
					return f_report('EEAromatic',x);
				}else if(action == "Preview XML"){
					return f_report('EEAromatic',x);
				}else if(action == "Check XML"){
					return f_report('EEAromatic',x);
				}else if(action == "Submit to EEMS"){
					return f_report('EEAromatic',x);
				}
 
			}else{
				alert('Please insert the Start / End date');
			}
		}else{
		    var lform = top.ReportInputFRM.document.all.ReportPreview;
	 	 	var oil_gas = false;
			if(lform.optionNames){
			//	var ta = lform.document.getElementsByName("optionNames");
				for(var i = 0; i < ta.length; i++){
					if (ta[i].value == "OperatingHours"){
						var operatHLoc = i;
						oil_gas = true;
					}else if(ta[i].value =="operatenum"){
						var operatNLoc = i;
					}
				}
			}
			/// code to check leep year
			var yearH = document.getElementById("ReportYear").value;
			var hours_in_year = 0;
			var is_leep = false;
			if(( yearH % 4) == 0){
				if(((yearH % 100) == 0) && ((yearH % 400) != 0)){
					hours_in_year = 8760;
				}else{
					hours_in_year = 8784;
					is_leep = true;
				}
			}else{
				hours_in_year = 8760;
			}	
 
		// code to check if the operatHours not grater than max amount of hours in year	
			var turbine_hours = 0;
			var check_ok = true;
			if(oil_gas){
			//	alert(lform.document.getElementsByName(ta[operatNLoc].value)[0].value);
				var turbineNrs = lform.document.getElementsByName(ta[operatNLoc].value)[0].value.split(",");
 
				for(var i = 0; i < (turbineNrs.length-1); i++){
					//	alert(turbineNrs[i] +" "+lform.document.getElementsByName(turbineNrs[i])[0].value);	
 
					turbine_hours = lform.document.getElementsByName(turbineNrs[i])[0].value;
 
					if(turbine_hours > hours_in_year){
						check_ok = false;	
					}
					 //sArgs += "&"+turbineNrs[i]+"=" +lform.document.getElementsByName(turbineNrs[i])[0].value; 
				}
			}
	 		if((action == "Overview") && (check_ok) ){
					return f_report('EEAromatic',x);
			}else if(action == "Preview XML"){
					return f_report('EEAromatic',x);
			}else if(action == "Check XML"){
				return f_report('EEAromatic',x);
			}else if(action == "Submit to EEMS"){
				return f_report('EEAromatic',x);
			}else if (!check_ok){
				alert("Please Check the hours input fields Max value for this year is "+hours_in_year);
			}else{
				alert('You have added in a new button method not supported by is_dates_set please uppdate the java script to support it');
			}
	    }
	}else{
		if(action == "Overview"){
			return f_report('EEAromatic',x);
		}else if(action == "Preview XML"){
			return f_report('EEAromatic',x);
		}else if(action == "Check XML"){
			return f_report('EEAromatic',x);
		}else if(action == "Submit to EEMS"){
			return f_report('EEAromatic',x);
		}
	} 
}	
--></script>
 </head>
 <body class="reg" >
 
 
  <form name="ReportPreview" id="ReportPreview" method="post" target="_blank" action="nems.jhtml?sda">
 
  	<input type="hidden" name="method" value="">
	<input type="hidden" name="InstIDs" value="" id="InstIDs">
   <div>
   <table border="0" style="border-collapse:collapse" >
	<colgroup>
		<col width="5" />
		<col width="70" />
		<col width="130" />
		<col width="130" />
 
		<col width="100" />
		<col width="100" />
		<col width="5" />
 	</colgroup>
   <tr><td></td>
 
 
 
	<td><b>Year</b></td> 
	   	<td><b>Half year</b></td>
   	   			<td><b>Event date(yyyy-mm-dd)</b></td><td><b>Sampled by</b></td><td><b>OPPC Permit Number</b></td>
 
 
   </tr>
   <tr>
   <td>
     	<img type="button" onclick="f_refresh_newInstallation('EEAromatic');" src="accounter/img/button_vertical_refresh.png" alt="refresh" title="refresh after change of installation" onmouseover="this.src='accounter/img/button_vertical_refresh_over.png'" onmouseout="this.src='accounter/img/button_vertical_refresh.png'" />
         	</td>
	<td>
      	<SELECT name="ReportYear" onchange="f_ajax_load_yearhours();" id="ReportYear">
      		      			<OPTION value="1999" >1999</OPTION>
 
		          			<OPTION value="2000" >2000</OPTION>
		          			<OPTION value="2001" >2001</OPTION>
		          			<OPTION value="2002" >2002</OPTION>
		          			<OPTION value="2003" >2003</OPTION>
		          			<OPTION value="2004" >2004</OPTION>
		          			<OPTION value="2005" >2005</OPTION>
 
		          			<OPTION value="2006" >2006</OPTION>
		          			<OPTION value="2007" >2007</OPTION>
		          			<OPTION value="2008" >2008</OPTION>
		          			<OPTION value="2009" >2009</OPTION>
		          			<OPTION value="2010"  selected="selected" >2010</OPTION>
		          			<OPTION value="2011" >2011</OPTION>
 
		          			<OPTION value="2012" >2012</OPTION>
		          			<OPTION value="2013" >2013</OPTION>
		          	</SELECT>
  	</td>
 
	  <td><SELECT name="ReportHalfYear"><Option value="1">1</Option><Option value="2">2</Option>  </SELECT></td>
 
 
	<td><input type="text" name="EventDate" /></td><td><input type="text" name="SampledBy" /></td><td><input type="text" name="oppcPermitNumber" /></td>
			<input type="hidden" name="optionNames" value="EventDate" />
			<input type="hidden" name="optionNames" value="SampledBy" />
			<input type="hidden" name="optionNames" value="oppcPermitNumber" />
 
          <tr>
       <td><div align="right" style="padding: 5px;"></div></td>
   </tr>
 
	<tr>
	<td></td>
 
 
	<td><input type="button" style="width:100%;" id="overview" name="overview" value="Overview" onclick="return is_dates_set('Overview',('EEAromatic'),0);"/></td>
    <td><input type="button" style="width:100%;" id="preview" name="preview" value="Preview XML" onclick="return is_dates_set('Preview XML',('EEAromatic'),1);"/></td>
        </tr>
	<tr><td><br/></td></tr>
	<tr>
	<td></td><td>EEMS user:</td>
	<td>
	Password:</td></tr>
 
	<tr><td></td><td>
	<input type="text" name="eemsuser" />
	</td><td>
	<input type="password" name="pswd" /></td>
 
</tr>
<tr>
	<td><div align="right" style="padding: 5px;"></div></td>
 
</tr>
<tr><td></td>
    <td><input type="button" style="width:100%;" id="check" name="check" value="Check XML" onclick="return is_dates_set('Check XML',('EEAromatic'),2);"/></td>
 
    <td><input  type="button" style="width:100%;" value="Submit to EEMS"  onclick="return is_dates_set('Submit to EEMS',('EEAromatic'),3);"/><br /></td> 
</tr>
 
<tr><td></td><td>Submit to test server: <input id="submitTest" name="submitTest" type="Checkbox" value="true"/></td></tr>
</table>
</div>
 
  </form>
 </body>
</html>

[franklin626]

j'ai finalement dégagé toutes les variables intermédiaires et refactorisé le tout.
ça passe partout à présent, mais je n'ai pas compris la différence.