JavaScript Forum

**TyR** · 02-23-2007, 01:56 PM

Hi, I'm working on this little script:

ajax.js:

Code:

var ajax_req = getXmlHttpRequestObject();

function getXmlHttpRequestObject() {
	if (window.XMLHttpRequest) {
		return new XMLHttpRequest();
	} else if(window.ActiveXObject) {
		return new ActiveXObject("Microsoft.XMLHTTP");
	} else {
		alert("XmlHttpRequest not supported");
	}
}

function ajax_load(x, y) {
	if (ajax_req.readyState == 4 || ajax_req.readyState == 0) {
		ajax_req.open("GET", x, true);
		ajax_req.onreadystatechange = function () { y };
		ajax_req.send(null);
	}
}

function ajax_output(x) {
	if (ajax_req.readyState == 4) {
		document.getElementById(x).innerHTML = ajax_req.responseText;
	}
}

HTML:

HTML Code:

<html>
    <head>
        <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
        <script src="engine/ajax.js" type="text/javascript"></script>
    </head>
    
    <body>

        <button id="moop">LOAD</button>
        <span id="result"></span>

        <script>
            document.getElementById("moop").onclick = function () { ajax_load('forsiden.php', ajax_output('result')); }
        </script>

    </body>
</html>

And it kinda works, but you have to press the button two times for the text to appear, and I need some help to understand why, and how I can fix this.

Could anyone point me in the right direction?

Thanks

**veridicus** · 02-24-2007, 05:52 PM

I don't know why exactly you're having that problem, but I would put the button into a form, set the onclick attribute directly in that tag, and take a look at the Prototype library.

**psychopsi** · 03:16 AM

It looks like this might be the problem:

Code:

if (ajax_req.readyState == 4 || ajax_req.readyState == 0) {

try this:

Code:

if (ajax_req.readyState == 4 && ajax_req.status == 200) {

you'll have to move the ajax_req.open to before that if block