Discussion :

[Raylemon]

Bonjour, je planche depuis plusieurs jours sur un problème que je n'arrive pas à résoudre.

J'ai écrit un script qui fait des appels via AJAX, pour envoyer plusieurs formulaires. Sous IE ça passe, mais sous Firefox, j'obtiens toujours la même erreur :

Erreur : form has no properties
Fichier source : http://127.0.0.1/Test/myth.js
Ligne : 7

la fonction qui pose problème est celle-ci :

Code :

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

1
2
3
4
5
6
7
8
function checkForm(form){
	var  i;
	var done=0;
	for (i=0;i<form.length;i++)//l'erreur se situe ici
		if (!form.elements[i].value)	done++;
	if (done==0) return true;
	else return false;
}

Cette fonction est appelée par celle-ci :

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
function addCat(){
	var form = document.getElementById("admCat");
	var resp = document.getElementById("admCatResp");
	var xhr= init();
	xhr.open("POST","server/admAddCat.php",true);
	if(checkForm(form)) {
		xhr.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
		xhr.send(getFormData(form));
		xhr.onreadystatechange = function() {
			if (xhr.readyState == 4) {
				if (xhr.responseXML.getElementsByTagName("Status")[0].text == "Add"){
					var id = xhr.responseXML.getElementsByTagName('Id')[0].text;
					var name = xhr.responseXML.getElementByTagName('Name')[0].text;
					new_element = new Option(name,id);
					form.cbIdCat.options[form.cbIdCat.length] = new_element;
					resp.innerHTML = "Catégorie n° "+xhr.responseXML.getElementsByTagName('Id')[0].text+" ajoutée";
				}
				else if (xhr.responseXML.getElementsByTagName("Status")[0].text == "Update")
					resp.innerHTML = "Catégorie n°"+xhr.responseXML.getElementsByTagName('Id')[0].text+" modifiée";
			}
			else resp.innerHTML= treatmentError;
			form.reset();
		}
	}
	else resp.innerHTML = errorMsg;
}

Pouvez-vous me dire ce qui ne vas pas ?
Merci d'avance,

[SpaceFrog]

elle fait quoi la focntion init() je suppose qu'elle initialise ajax ... mais le fait elle correctement pour IE et FIREFOX ?

[Raylemon]

Oups, pardon
A force de voir toujours les même lignes de code, je ne fais plus attention.
Oui, init() initialise ajax. Elle est correcte, puisque l'objet est créé. C'est juste sur le formulaire que ça coince

Code :

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

1
2
3
4
5
6
7
function init(){
    var xhr = null; 
    if(window.XMLHttpRequest) xhr = new XMLHttpRequest(); 
    else if(window.ActiveXObject) xhr = new ActiveXObject("Microsoft.XMLHTTP"); 
    else alert("Votre navigateur ne supporte pas les objets XMLHTTPRequest...");   
    return xhr; 
}

[SpaceFrog]

ça m'a pourtant l'air correct ...

essaye evc ce bout de code ci qui est éprouvé ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
    switch ( true ) {    
                    case (!!window.XMLHttpRequest):
                                            ReqObj=new XMLHttpRequest();
                                            break;
 
                    case  (!!window.ActiveXObject):
                                            ReqObj=new ActiveXObject("Microsoft.XMLHTTP");
                                            break;
 
                    default :                            alert("Votre navigateur ne supporte pas les objets XMLHTTPRequest...");
                      }

[Raylemon]

Merci, mais en essayant, le problème de change pas. Mon objet xhr est bien créé.
En fait, l'erreur proviendrait du formulaire. En faisant un alert dans ma fonction checkForm() (alert(form)), je remarque que sous IE, il renvoie un objet, mais sous Firefox, il renvoie NULL. Je comprends pas pourquoi

[SpaceFrog]

elle est lancée ou la focntion checkform() tu passes quoi en paramètre ? this.form ou document.forms[0] ?

[Raylemon]

CheckForm est appelée dans la fonction AddCat (ligne 6 de la fonction AddCat)
elle reçoit en paramètre le formulaire. Ce formulaire est récupéré via document.getElementById('AdmCat');

[SpaceFrog]

arf dsl pas vu ...
essaye

Code :

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

var monform = document.forms[0];

puis

Code :

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

if(checkForm(monform))

[Raylemon]

Ah ben merci, ça marche !!
Mais je me demande pourquoi la méthode document.getElementById ne fonctionne pas. J'ai lu que c'était la meilleure méthode à utiliser, pourtant...

[SpaceFrog]

ben il reconnait pas que c'est form avec l'id ... encore une curiosté ...

[Raylemon]

Ok, merci beaucoup de ton aide !