Script per banner

Ciao a tutti!

Sto creando uno script per inserire i banner nel proprio sito.

Questo è il codice da incollare in una pagina:

<script type="javascript" src="http://allo.altervista.org/_altervista_ht/banner.js"></script>

Questo è banner.js:

document.write('<s'+'cript type="text/javascript" src="http://allo.altervista.org/_altervista_ht/banner.php?size=small"></s'+'cript>');

e infine banner.php

<?
if($size=="small"){
echo "document.write('<a href="allo.altervista.org"><img src="http://allo.altervista.org/_altervista_ht/banner.gif" alt="Il mio sito"></a>')";
}
?>

Ma non funziona.
Come mai?

<?
if($_GET['size']=="small"){
echo "document.write('<a href=\"allo.altervista.org\"><img src=\"http://allo.altervista.org/_altervista_ht/banner.gif\" alt=\"Il mio sito\"></a>')";
}
?>

questo per quanto riguarda il php, il js non lo so ....

niente, non ne vuole sapere...

La parte in php è giusta, non ci piove.
Secondo me è il js a dar problemi ...

prova ad inserire nella pagina questo:

<script type="javascript" src="http://allo.altervista.org/_altervista_ht/banner.php?size=small"></script>

invece del doppio js

l'errore sta nel codice js generato dal file in php
mancano i tag script

Edit:
Probabilmente non funzionerà neanke mettendo i tag script
forse sarebbe meglio usare un iframe invece di includere il file php come un file js

Sono riuscito.
Grazie per gli aiuti!

[mythologia]

Potresti postare come? (E' utile per tutti)

secondo me ha tolto document.write ecc xke bastava mettere echo"<a href=\"link\">testo</a> e funzionava

[heracleum]

l'errore sta nel codice js generato dal file in php
mancano i tag script

Edit:
Probabilmente non funzionerà neanke mettendo i tag script
forse sarebbe meglio usare un iframe invece di includere il file php come un file js

Infatti non era quello il problema.
Non so come abbia risolto allo...
anzi, ne approfitto per ripetere per la centounesima volta che sarebbe molto gentile da parte di chi chiede aiuto e trova una sua soluzione, scrivere COME ha risolto: sia per l'utilità del thread (che altrimenti termina "nella nebbia") e gentile anche nei confronti di chi ha aiutato.

Ad ogni modo, visto che allo non ha ritenuto utile dire come ha risolto,
vi dico come secondo me doveva essere risolto correttamente:

ci siete andati molto vicini quando Inverno ha corretto le doppievirgolette nella stringa php...
sarebbe bastato solo aggiungere in cima al codice php la dichiarazione che il contenuto restituito è del tipo text/javascript, come sarebbe giusto scrivere accanto ad ogni tag SCRIPT (attenzione type="text/javascript" e non type="javascript" come appare in uno dei due file js.
Dicevo sarebbe bastato:

1) inserire la riga php:

Header("Content-type: text/javascript"); 

e
2) correggere appunto quel <script type="javascript"
<script type="text/javascript"

Scusate, ma non avevo proprio tempo.
(Ho messo tutto in _altervista_ht)

Script da inserire nella pagina:

<script src="http://allo.altervista.org/_altervista_ht/banner120x60.js"></script>

banner120x60.js:

document.write('<s'+'cript type="text/javascript" src="http://allo.altervista.org/_altervista_ht/banner.php?size=120x60"></s'+'cript>');

e banner.php:

<?
if($size=="120x60"){
echo "document.write('<a href=\"http://allo.altervista.org/index.php\" target=\"_blank\"><img border=\"0\" src=\"http://allo.altervista.org/_altervista_ht/120x60.gif\" alt=\"Il mio sito!\"></a>')";
}
?>

Assicuro che funziona!
Praticamente ho usato la stessa struttura del .js della top100 di av, modificandolo con il mio sito e inserendo la pagina PHP al posto di quella cgi.

Scusate ancora!

[heracleum]

Sì ma sei sicuro che funziona su più browser? per es. su Mozilla?
di solito IE è molto "generoso" mentre Mozilla tiene molto ai Content-Type, quindi se nel tag SCRIPT si indica

type="text/javascript"

vuol dire che il banner.php, oltre a restituire codice javascript deve dichiarare per correttezza di restituire contenuto "text/javascript" come ho indicato sopra al punto 1).
Come minimo anche se funzionasse genereresti un sacco di "warning" sul content-type atteso (text/javascript) mentre non indicando nulla dovrebbe restituire se non erro text/html.

A me funziona su Firefox e a un mio amico su Mozilla, pure.
Per Mac (ad es. Safari), poi, non so...

[heracleum]

ok,
Hai guardato nella console javascript?
sicuramente avrai i warning di cui sopra (almeno su Mozilla, di solito segnala 'ste cosacce)

Comunque, anche non fosse, non fa male fare le cose perbenino no?

Quindi nella pagina PHP dovrei inserire

Header("Content-type: text/javascript"); 

giusto?

[heracleum]

yes! giusto
e fai la cosa pulita pulita