Discussion :

[foxeye]

Bonjour,

Je suis confronté à deux problèmes afin que mon extension Firefox soit validé par Mozilla.

Le premier est la suppression des eval(), voici le code JS :

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
i = 0;
var makeCD = "this.treeView.childData = {";
lastURL = "";
while(this.stickiesData[i])
{
	if(this.stickiesData[i][8] != lastURL)
	{
		if(lastURL != "")
			makeCD += "],";
		makeCD += "\"" + this.stickiesData[i][8] + "\": [" + i;
	}
	else
	{
		makeCD += "," + i;
	}
 
	lastURL = this.stickiesData[i][8];
	i++;
}
makeCD += "]};";
 
eval(makeCD);

Et ensuite Mozilla demande d'ajouter un namespace afin d'éviter tout conflit avec une autre extension.
Le problème c'est que cette extension à été codé pour la première fois en 2006 et j'ai effectué une migration afin de la rendre valide avec Firefox > 3.
Convertir les 4000 lignes en object serait beaucoup trop long, existe t-il une solution alternative ?

Merci d'avance pour vos réponses.

[SpaceFrog]

Code :

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

eval('('+makeCD+')');

[foxeye]

Merci pour votre réponse mais une nouvelle erreur apparaît avec cette syntaxe :

Code :

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

1
2
3
4
Erreur*: missing ) in parenthetical
 
Code Source*:
 (this.treeView.visibleData = [["perdu.com", true, false, 0]];)

[SpaceFrog]

on peut voir un exemple de chaine complète que tu essayes d'evaluer ?

par exemple ceci focntionne:

Code :

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

1
2
3
var foo='{bar:["perdu.com", true, false, 0]}'
var monjson=eval('('+foo+')')
alert(monjson.bar[0])

[foxeye]

Ok c'est bon. C'est le point virgule de la fin qui posait problème.

Ne reste plus que le problème du namespace. Voici la partie du mail de Mozilla :
"In order to prevent conflicts with other add-ons that may be installed by users, you need to wrap your 'loose' variables and functions within a JavaScript object."

Pour les eval() ça va passer comme ça ?

Merci

[SpaceFrog]

Aucune idée, je ne me suis pas encore lancé dans la creation d'addons ...

à priori il faut éviter des variables seules afin de prevenir tout conflit avec d'autres variables globales ...

sons doute un peu plus d'infios sur le namesapcing en js par là
http://www.dustindiaz.com/namespace-your-javascript/