readystatechange, getElementByID help needed

[mikemcleod]

Hello everyone,

Long time lurker, first time poster.

I'm trying to implement an AJAX inline editor and am having some trouble. I want to use this editing field several times on
the same page, and was hoping to make the JavaScript reusable, passing
it the name of the fields that need to change, but I'm having some
trouble. In particular, I'm having trouble here:

------------
function sndReq(action) {
http.open('get', action);
http.onreadystatechange = handleResponse;
http.send(null);
}

function handleResponse() {
if(http.readyState == 4){
var replaceText = document.getElementById(name_textbox).value;
document.getElementById('name_displaybox_contents' ).innerHTML = replaceText;
document.getElementById('name_displaybox').style.d isplay = '';
document.getElementById('name_editingbox').style.d isplay = 'none';
}
}
------------

My thought was that at the <http.onreadystatechange = handleResponse;>
line I could pass the value of the field element I'm using on to the
handleResponse function and then use it to swap out the
getElementByIds so that I wouldn't need to replicate all this code for
the different chunks I'm swapping out. However, the http.readyState
is making things difficult for me; for some reason, if I pass a
variable on to handleResponse, the "if (http.readyState == 4)" doesn't get
tripped.

Does this make any sense? I don't know, because it doesn't make sense to me either. If you could help me out I'd be greatly in your debt!

Mike

[epots9]

Hello everyone,

Long time lurker, first time poster.

I'm trying to implement an AJAX inline editor and am having some trouble. I want to use this editing field several times on
the same page, and was hoping to make the JavaScript reusable, passing
it the name of the fields that need to change, but I'm having some
trouble. In particular, I'm having trouble here:

------------
function sndReq(action) {
http.open('get', action);
http.onreadystatechange = handleResponse;
http.send(null);
}

function handleResponse() {
if(http.readyState == 4){
var replaceText = document.getElementById(name_textbox).value;
document.getElementById('name_displaybox_contents' ).innerHTML = replaceText;
document.getElementById('name_displaybox').style.d isplay = '';
document.getElementById('name_editingbox').style.d isplay = 'none';
}
}
------------

My thought was that at the <http.onreadystatechange = handleResponse;>
line I could pass the value of the field element I'm using on to the
handleResponse function and then use it to swap out the
getElementByIds so that I wouldn't need to replicate all this code for
the different chunks I'm swapping out. However, the http.readyState
is making things difficult for me; for some reason, if I pass a
variable on to handleResponse, the "if (http.readyState == 4)" doesn't get
tripped.

Does this make any sense? I don't know, because it doesn't make sense to me either. If you could help me out I'd be greatly in your debt!

Mike

Hi mikemcleod welcome to TSDN,
since it wasn't posted i think u forgot to do this:

var http;
function sndReq(action) 
{
http = GetXmlHttpObject();
http.open('get', action);
http.onreadystatechange = handleResponse;
http.send(null);
}
function GetXmlHttpObject()
{
    var xmlHttp=null;
    try
    {
        // Firefox, Opera 8.0+, Safari
        xmlHttp=new XMLHttpRequest();
    }
    catch (e)
    {
        // Internet Explorer
        try
        {
            xmlHttp=new ActiveXObject("Msxml2.XMLHTTP");
        }
        catch (e)
        {
            try
            {
                xmlHttp=new ActiveXObject("Microsoft.XMLHTTP");
            }
            catch(e)
            {
                xmlHttp=null;
            }
        }
    }
    return xmlHttp;
}
 

good luck

[mikemcleod]

Hi epots9,

Thanks for your message. I didn't post the part of my code where I open the XMLHttpRequest because I didn't think it was directly relevant to my problem.

The problem is that, when I modify the code I posted earlier to pass a variable on to the handleResponse function like this:

-----------------------
function sndReq(action, formelement) {
http.open('get', action);
http.onreadystatechange = handleResponse(formelement);
http.send(null);
}

function handleResponse(divVariable) {
if(http.readyState == 4){
var replaceText = document.getElementById(divVariable).value;
document.getElementById(divVariable).innerHTML = replaceText;
}
}
-----------------------

the "if(http.readyState == 4)" condition does not get tripped - when I don't pass the variable, as in the first script example I posted, the script works just fine, but I have to put the full name of the div into the getElementById function.

Totally baffled. Any help is much appreciated!

Mike

[epots9]

Hi epots9,

Thanks for your message. I didn't post the part of my code where I open the XMLHttpRequest because I didn't think it was directly relevant to my problem.

The problem is that, when I modify the code I posted earlier to pass a variable on to the handleResponse function like this:

-----------------------
function sndReq(action, formelement) {
http.open('get', action);
http.onreadystatechange = handleResponse(formelement);
http.send(null);
}

function handleResponse(divVariable) {
if(http.readyState == 4){
var replaceText = document.getElementById(divVariable).value;
document.getElementById(divVariable).innerHTML = replaceText;
}
}
-----------------------

the "if(http.readyState == 4)" condition does not get tripped - when I don't pass the variable, as in the first script example I posted, the script works just fine, but I have to put the full name of the div into the getElementById function.

Totally baffled. Any help is much appreciated!

Mike

what is the variable "formelement"? u're receiving it in the sndReq function, what is it? posts its html code and how your sedning it to the sndReq function.