Discussion :

[Are-no]

Bonjour,

je me suis fait le script suivant :

Code :

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

1
2
3
4
5
6
7
8
<script type="text/javascript"><!--
benight_dpt = '60';
benight_lieu = '0';
benight_height = '600';
benight_width = '400';
//-->
</script>
<script src="http://www.be-night.com/api/agenda.js" type="text/javascript"></script>

et le script de la page http://www.be-night.com/api/agenda.js

Code :

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

1
2
3
4
5
6
document.write('
<iframe width="' + benight_width + '" height="' + benight_height + '" frameborder="0" allowtransparency="true" 
hspace="0" vspace="0" marginheight="0" marginwidth="0" 
src="http://www.be-night.com/api/agenda/aff.php?dpt=' + benight_dpt + '&lieu=' + benight_lieu + '">
</iframe>
');

Mais le problème (et uniquement sous IE) c'est qu'il me dit que la variable "benight_width" est indéfini

comment faire ?

[E.Bzz]

Bonjour,
à quel moment s'exécute le document.write() ?
Tu as essayé de mettre les "var" devant tes déclarations ?
De passer les déclarations dans le ".js" ?

A+

[Are-no]

Pour etre un peu plus précis, le 1er code ce trouve sur divers site client et le 2eme ce trouve sur mes serveurs.

Les var devant ne change rien a l'affaire.

Et pour ce qui est de passer les déclaration dans le .js, autant fournir directement le 2eme code au client, ce qui est moin lisible.

Pour information, je me suis bassé sur le script fourni pour les pub Google

[Are-no]

Apres avoir regarder le code de chez goolge, me voilà un peu perdu...

Voila le code qu'il nous donne pour mettre sur nos pages :

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
	<script type="text/javascript"><!--
	google_ad_client = "pub-xxxxxxxxxxxxxxx";
	google_alternate_color = "000000";
	google_ad_width = 728;
	google_ad_height = 90;
	google_ad_format = "728x90_as";
	google_ad_type = "text_image";
	google_ad_channel = "";
	google_color_border = "333333";
	google_color_bg = "000000";
	google_color_link = "FFFFFF";
	google_color_text = "FFFFFF";
	google_color_url = "FFFFFF";
	google_ui_features = "rc:10";
	//-->
	</script>
	<script type="text/javascript"
	  src="http://pagead2.googlesyndication.com/pagead/show_ads.js">
	</script>

jusque là, ca va.

et voilà le lien vers leur page http://pagead2.googlesyndication.com/pagead/show_ads.js

et la j'avoue que je ne suis pas tout...

[E.Bzz]

et la j'avoue que je ne suis pas tout...

Visiblement, ils passent par un cookie pour transmettre les valeurs : regarde la déclaration de la variable "c" en début de script.
Tu retrouves ensuite dans le code

Code :

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

c.google_ad_width

A+

[E.Bzz]

à quel moment s'exécute le document.write() ?

Plus précisément : est-il intégré à une fonction ?
Auquel cas, il serait préférable d'appeler (à partir de la page parent) cette fonction en passant les 4 valeurs en paramètres ...

A+

[Are-no]

Le script http://www.be-night.com/api/agenda.js ne comporte que le document.write()

Pour l'idée du cookie, c'est ce que j'ai pensé au début mais je ne vois pas a quel moment il on mis en place les cookie car le seul code fourni par google pour mettre sur nos page est celui que j'ai donnée un peu avant.

[E.Bzz]

je ne vois pas a quel moment il on mis en place les cookie car le seul code fourni par google pour mettre sur nos page est celui que j'ai donnée un peu avant.

Je ne fais que constater

A+

[Are-no]

Bon, j'ai trouvé le problème et je l'annonce quand même avec un peu de honte...

le 1er script été introduit dans le code de la page en passant avant par la fonction php nl2br(), ce qui me rajouter des <br /> a chaque fin de ligne. Firefox passé a travers mais pas IE. Le code marche donc tres bien quand il ets mis en direct dans le code... Jim_Nastiq avez vu bon

[Jim_Nastiq]

hum! chez moi cela fonctionne sur IE 6 et sur FF 2

[d-Rek]

Aucun rapport à part peut être si tu fais des calculs mais bon juste pour info, tu peux aussi mettre tes valeurs sans quote car ce sont des integers.

Code :

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

var toto = 60;