erreur Internet Explorer 6 - 8

Version imprimable

06/08/2012, 16h18
chochatown

2 pièce(s) jointe(s)

erreur Internet Explorer 6 - 8

Bonjour,

pour les experts de IE :)

J'ai une erreur sur ma page seulement quand j'y accede avec IE.

les messages d'erreurs sont en peices jointes. Ce qui est ettonant, la ligne d'erreur est dans une fonction reconnu "getElementsByClassname" qui est valide sur tt les navigateurs web...

j'ai l'impression que le deboggueur ne me donne pas la bonne information. Et ce qui est sur c'est que le message d'erreur manque beaucoup d'informations.

Merci pour votre aide! :D
06/08/2012, 16h26
arnogues

Non cette fonction n'existe pas dans IE6/7/8 donc je te conseille de la coder ou de passer par une librairie genre jquery
06/08/2012, 16h37
chochatown

C'est ce que je suis entrain de faire, l'erreur est dans le code de la fonction que j'ai ajoute, et c'est cette fonction que j'utilise. la ligne d'erreur est dans le code la fonction que j'ai du ajouter puisqu'elle est pas reconnu dans IE. dsl, je pense que j'ai mal explique au debut. le code de la fonction etait teste par son auteur sur tt les navigateurs que je voulais dire et non pas la fonction qui est reconnu.
06/08/2012, 17h00
NoSmoking

Bonjour,
il eut mieux valu joindre le code de la fonction plutôt qu'une image, marci d'y penser par la suite.

- ne pas omettre de déclarer les variables à l'aide du mot clé var.
- ne pas utiliser des mots connu du langage comme match.

essaies de changer le nom de cette variable.

Voila je viens de modifier lal fonction, mais autre erreur intervinne apres cette modification! :

Citation:

ReferenceError: current is not defined
[Break On This Error]

current;

Avant :

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
    /*
            Developed by Robert Nyman, http://www.robertnyman.com
            Code/licensing: http://code.google.com/p/getelementsbyclassname/
        */    
        s.getElementsByClassName = function (className, tag, elm){
			var getElementsByClassName = function(){}; // limit scope of function
 
            if (document.getElementsByClassName) {
                getElementsByClassName = function (className, tag, elm) {
                    elm = elm || document;
                    var elements = elm.getElementsByClassName(className),
                        nodeName = (tag)? new RegExp("\\b" + tag + "\\b", "i") : null,
                        returnElements = [],
                        current;
                    for(var i=0, il=elements.length; i<il; i+=1){
                        current = elements[i];
                        if(!nodeName || nodeName.test(current.nodeName)) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            else if (document.evaluate) {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    var classes = className.split(" "),
                        classesToCheck = "",
                        xhtmlNamespace = "http://www.w3.org/1999/xhtml",
                        namespaceResolver = (document.documentElement.namespaceURI === xhtmlNamespace)? xhtmlNamespace : null,
                        returnElements = [],
                        elements,
                        node;
                    for(var j=0, jl=classes.length; j<jl; j+=1){
                        classesToCheck += "[contains(concat(' ', @class, ' '), ' " + classes[j] + " ')]";
                    }
                    try    {
                        elements = document.evaluate(".//" + tag + classesToCheck, elm, namespaceResolver, 0, null);
                    }
                    catch (e) {
                        elements = document.evaluate(".//" + tag + classesToCheck, elm, null, 0, null);
                    }
                    while ((node = elements.iterateNext())) {
                        returnElements.push(node);
                    }
                    return returnElements;
                };
            }
            else {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    var classes = className.split(" "),
                        classesToCheck = [],
                        elements = (tag === "*" && elm.all)? elm.all : elm.getElementsByTagName(tag),
                        current,
                        returnElements = [],
                        match;
                    for(var k=0, kl=classes.length; k<kl; k+=1){
                        classesToCheck.push(new RegExp("(^|\\s)" + classes[k] + "(\\s|$)"));
                    }
                    for(var l=0, ll=elements.length; l<ll; l+=1){
                        current = elements[l];
                        match = false;
                        for(var m=0, ml=classesToCheck.length; m<ml; m+=1){
                            match = classesToCheck[m].test(current.className);
                            if (!match) {
                                break;
                            }
                        }
                        if (match) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            return getElementsByClassName(className, tag, elm);
        };

apres enlever les var et les match :

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
        /*
            Developed by Robert Nyman, http://www.robertnyman.com
            Code/licensing: http://code.google.com/p/getelementssbyclassname/
        */    
        s.getElementsByClassName = function (className, tag, elm){
			getElementsByClassName = function(){}; // limit scope of function
 
            if (document.getElementsByClassName) {
                getElementsByClassName = function (className, tag, elm) {
                    elm = elm || document;
                    elementss = elm.getElementsByClassName(className),
                        nodeName = (tag)? new RegExp("\\b" + tag + "\\b", "i") : null,
                        returnElements = [],
                        current;
                    for(iount=0, il=elementss.length; iount<il; iount+=1){
                        current = elementss[iount];
                        if(!nodeName || nodeName.test(current.nodeName)) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            else if (document.evaluate) {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    classes = className.split(" "),
                        classesToCheck = "",
                        xhtmlNamespace = "http://www.w3.org/1999/xhtml",
                        namespaceResolver = (document.documentElement.namespaceURI === xhtmlNamespace)? xhtmlNamespace : null,
                        returnElements = [],
                        elementss,
                        node;
                    for(jount=0, jl=classes.length; jount<jl; jount+=1){
                        classesToCheck += "[contains(concat(' ', @class, ' '), ' " + classes[jount] + " ')]";
                    }
                    try    {
                        elementss = document.evaluate(".//" + tag + classesToCheck, elm, namespaceResolver, 0, null);
                    }
                    catch (e) {
                        elementss = document.evaluate(".//" + tag + classesToCheck, elm, null, 0, null);
                    }
                    while ((node = elementss.iterateNext())) {
                        returnElements.push(node);
                    }
                    return returnElements;
                };
            }
            else {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    classes = className.split(" "),
                        classesToCheck = [],
                        elementss = (tag === "*" && elm.all)? elm.all : elm.getElementsByTagName(tag),
                        current,
                        returnElements = [],
                        matche;
                    for(kount=0, kl=classes.length; kount<kl; kount+=1){
                        classesToCheck.push(new RegExp("(^|\\s)" + classes[kount] + "(\\s|$)"));
                    }
                    for(lount=0, ll=elementss.length; lount<ll; lount+=1){
                        current = elementss[l];
                        matche = false;
                        for(mount=0, ml=classesToCheck.length; mount<ml; m+=1){
                            matche = classesToCheck[mount].test(current.className);
                            if (!matche) {
                                break;
                            }
                        }
                        if (matche) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            return getElementsByClassName(className, tag, elm);
        };

pourquoi avoir modifié l'original ?

Code:

1
2
3
var getElementsByClassName = function (className, tag, elm){
	if (document.getElementsByClassName) {
		getElementsByClassName = function (className, tag, elm) {

devient

Code:

1
2
3
4
s.getElementsByClassName = function (className, tag, elm){
	var getElementsByClassName = function(){}; // limit scope of function
        if (document.getElementsByClassName) {
               getElementsByClassName = function (className, tag, elm) {

06/08/2012, 22h09
chochatown

C'est juste pour des tests. Je limite la fonction. Mais, là mon erreur s'affiche dans les deux cas. Cette ligne existe ou non... :cry: Je n'ai fait que enlever les var et changer le nom de match et ça donne une nouvelle erreur. :aie:
06/08/2012, 22h33
chochatown

Et voila un Warning de Closure-compiler :

Citation:

JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2870 character 0
currente;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2889 character 0
elementss;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2890 character 0
node;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2913 character 0
currente;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2915 character 0
matche;
07/08/2012, 10h32
NoSmoking

Ce qui m'interpèle le plus c'est le s.getElementsByClassName = ..., il faudrait voir le contexte dans lequel tu l'utilises mais je pense qu'il faut que tu reviennes à l'utilisation "standard" de la fonction, sans modification, il me semble que cette fonction à déjà fait ses preuves.

Finalement, le bug est resolu.

Pour des raisons inconnu lors de la comparaison :

Code:

match = classesToCheck[m].test(current.className)

Donc j'ai modifié cette ligne pour verifier les objets et leurs types comme suit :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 //match = classesToCheck[m].test(current.className); tmp1 = classesToCheck[m]; if(typeof current=='object') { if(typeof current.className=='string') { tmp2 = current.className; cmpr = tmp1.test(tmp2); if(cmpr == true) match = true; } }
Resolu! IE sucks!:calim2: :lol::lol: