un aiutino con una funzione (location.replace,window.close)

[The Captain]

Qualcuno mi dice cos'ha che non va questa funzione?

function fuckUpthisWindow(){
    window.opener = self;
    if(document.innerWidth <= 792 || document.documentElement.clientWidth <= 792){
        if(!window.close()){
                document.location.replace = ('index.php?act=Login');
        }
    } else {
        document.location.replace = ('index.php');
    }
}
window.setTimeout('fuckUpthisWindow()', 3000); 

L'avevo scritta, funzionava, poi ho modificato qualcosa tempo fa e adesso non va più.. boh, io non vedo errori.
Grazie

[artorius]

beh, document.location.replace non esiste...
devi fare document.location.href

comunque perché fai if(!window.close()){ ?

close() ha un ritorno void...

[The Captain]

Originariamente inviato da artorius
beh, document.location.replace non esiste...
devi fare document.location.href

ma che dici, certo che esiste location.replace.
La differenza tra location.href e location.replace è che location.href è come se caricasse una nuova pagina, lasciando la pagina precedente nel "Back" del browser, location.replace no (per questo è consigliato per evitare un ciclo di redirect inutile e dannoso)

http://www.bloggingdeveloper.com/pos...t-Methods.aspx

comunque perché fai if(!window.close()){ ?
close() ha un ritorno void...

perchè sono ignorante

chiudi = return window.close();
if chiudi false bla bla bla?

Comunque non è quello il problema.

[artorius]

ah, certo...
Ma replace è un methodo non una proprietà...

[The Captain]

Originariamente inviato da artorius
ah, certo...
Ma replace è un methodo non una proprietà...

...ovvero?

ho risolto con:

<script type="text/javascript">
<!--
function fuckUpthisWindow(){
    window.opener = self;
    if((document.innerWidth && document.innerWidth <= 792) || (document.documentElement.clientWidth && document.documentElement.clientWidth <= 792)){
        if(!window.close()){
            document.location.href=\'index.php?act=Login\';
        }
    } else {
        document.location.href=\'index.php\';
    }
}
window.setTimeout(\'fuckUpthisWindow()\', 3000);
//-->
</script> 

ma fino ad adesso location.replace ha sempre funzionato a meraviglia, mbah

[artorius]

function fuckUpthisWindow(){
    window.opener = self;
    if(document.innerWidth <= 792 || document.documentElement.clientWidth <= 792){
        if(!window.close()){
                document.location.replace('index.php?act=Login');// non document.location.replace = ('index.php?act=Login')
        }
    } else {
        document.location.replace('index.php');// non document.location.replace = ('index.php')
    }
}
window.setTimeout('fuckUpthisWindow()', 3000); 

E' un metodo non una proprietà

[The Captain]

Originariamente inviato da artorius
non document.location.replace = ('index.php?act=Login')

si, me ne sono accorto adesso.

Ma è un semplice errore di sintassi, ho dimenticato di togliere il = quando testavo location.href e location.replace

Grazie e ciao