Need some javascript help

[Christopher]

So I'm trying to make it that when reload=1 is added to the URL, the page refreshes upon doing this one action. I know it works if I don't check if reload=1. But I can't figure out how to check.
BTW, I don't know a thing about javascript. I did this by copying some code

Here is what I have. But it doesn't work.

var queryString = url.replace(/^[^\?]+\??/,'');
var params = tb_parseQuery( queryString );

if(params['reload'] == 1){
	location.reload(true);
}

Thanks for any help! There might be a credits reward

 

[woiwky]

Try using this instead:

if (location.search.match('reload=1')) {
    location.search = location.search.replace(/&?reload=1/, '');
}

 

[Christopher]

Nope.
I am able to get parameters from the URL.

TB_WIDTH = (params['width']*1) + 30 || 630; //defaults to 630 if no paramaters were added to URL
TB_HEIGHT = (params['height']*1) + 40 || 440; //defaults to 440 if no paramaters were added to URL

That works.

 

[woiwky]

Oh so you *want* to use the params variable? Well in that case I need to see the definiton for the url variable.

 

[Christopher]

Not quite sure want you want to see.
You can view the full code (without my attempted refresh thing) at http://jquery.com/demo/thickbox/

 

[woiwky]

Alright, so you're placing that code inside of the tb_show() function? At which part exactly? Right after this code?

var queryString = url.replace(/^[^\?]+\??/,'');
var params = tb_parseQuery( queryString );

 

[Christopher]

I put it in tb_remove()

function tb_remove() {
 	$("#TB_imageOff").unbind("click");
	$("#TB_closeWindowButton").unbind("click");
	$("#TB_window").fadeOut("fast",function(){$('#TB_window,#TB_overlay,#TB_HideSelect').trigger("unload").unbind().remove();});
	$("#TB_load").remove();
	if (typeof document.body.style.maxHeight == "undefined") {//if IE 6
		$("body","html").css({height: "auto", width: "auto"});
		$("html").css("overflow","");
	}
	document.onkeydown = "";
	document.onkeyup = "";
	if(params['reload'] == 1){
		location.reload(true);
	}
	return false;
}

 

[woiwky]

Ah, that's most likely your problem then since params isn't defined there.

Try changing it to this:

function tb_remove() {
     $("#TB_imageOff").unbind("click");
    $("#TB_closeWindowButton").unbind("click");
    $("#TB_window").fadeOut("fast",function(){$('#TB_window,#TB_overlay,#TB_HideSelect').trigger("unload").unbind().remove();});
    $("#TB_load").remove();
    if (typeof document.body.style.maxHeight == "undefined") {//if IE 6
        $("body","html").css({height: "auto", width: "auto"});
        $("html").css("overflow","");
    }
    document.onkeydown = "";
    document.onkeyup = "";
    var queryString = location.href.replace(/^[^\?]+\??/,'');
    var params = tb_parseQuery( queryString );
    if(params['reload'] == 1){
        location.reload(true);
    }
    return false;
}

 

[Christopher]

Still not working.

 

[woiwky]

Try adding 'alert(location.search);' right before 'var queryString...'. If you get no alert box or if you don't see reload=1 in the message, then the code isn't the problem. I just wanna narrow down exactly what the problem is.

 

[Christopher]

When I put it in the tb_remove() function, I get a blank alert.

 

[woiwky]

Well that means that for some reason the current URL has no query. Does the alert box show on page load or when you click a link? How exactly are you adding reload=1 to the query of the URL?

 

[Christopher]

You can see it at http://dev.fyepro.com
You'll find the link under User Menu titled Chat.

The alert box shows up when I close it.

 

[woiwky]

It seems to work fine for me. If I go to

http://dev.fyepro.com/index.php?reload=1

and then click on Chat and close it, it alerts with '?reload=1' and reloads the index page after that. Perhaps you have security settings blocking JS from reloading pages?

 

[Christopher]

Odd.
What I was trying to do is set the reload setting in the URL that the Chat link points to. Like how the height and width ones work.

 

[woiwky]

Ah, I see. I think the best way to do that would be by doing this in the tb_show() function:

var queryString = url.replace(/^[^\?]+\??/,'');
var params = tb_parseQuery( queryString );
window['queryParams'] = params;

So that it creates a global variable named queryParams. And then change the tb_remove() function to:

function tb_remove() {
     $("#TB_imageOff").unbind("click");
    $("#TB_closeWindowButton").unbind("click");
    $("#TB_window").fadeOut("fast",function(){$('#TB_window,#TB_overlay,#TB_HideSelect').trigger("unload").unbind().remove();});
    $("#TB_load").remove();
    if (typeof document.body.style.maxHeight == "undefined") {//if IE 6
        $("body","html").css({height: "auto", width: "auto"});
        $("html").css("overflow","");
    }
    document.onkeydown = "";
    document.onkeyup = "";
    if (queryParams['reload'] == 1){
        location.reload(true);
    }
    return false;
}

I believe that should work fine.

 

[Christopher]

It works!
Thank you!!!!!!!!!!!!!!

500 credits heading your way

 

[woiwky]

No problem. Thanks for the credits ;-)

 

[themasterrocker]

I don't really understand what you're trying to do but i can attempt to help.
To what i can see woiwky is leading you the right way. Is the refresh=1 nesarcary if not why have it there why not have it without because then it refreshes?