onreadystatechange NOT working in IE!!!!

[avance]

hello. i have a script that loads content into a div. it works in firefox but not IE. i have debugged and determined where it stops working, i just don't know why it ain't working. here is the code:

Code: Select all

function ajaxpage(url, containerid){
var page_request = false
if (window.XMLHttpRequest) // if Mozilla, safari etc
page_request = new XMLHttpRequest()
else if (window.ActiveXObject){ // if IE
try {
page_request = new ActiveXObject("Msxml2.XMLHTTP")
} 
catch (e){
try{
page_request = new ActiveXObject("Microsoft.XMLHTTP")
}
catch (e){}
}
}
else
return false

// stops working here
page_request.onreadystatechange=function(){
loadpage(page_request, containerid)
//alert('test');
}
page_request.open('GET', url, true)
page_request.send(null)
}

can anybody provide a solution please.

thanks in advance

[Rakuli]

If it's working in FF and gets up to readystatechange in IE I would think it's not your ajax function.. try sending an empty string rather than NULL in your send request as well

[avance]

thanks. i tried that but it still didn't work. i ain't too good with javascript so i don't know where to go next. any other solutions would be appreciated.

thanks

[Rakuli]

Can you post the code that is the loadPage function?... I assume you have tested at each part of the exception to make sure the XHTTP request is actually be processed by IE activeX?

[avance]

ok. here's the entire script:

Code: Select all

var loadedobjects=""
var rootdomain="<A href="http://"+window.location.hostname">http://"+window.location.hostname
function ajaxpage(url, containerid){
var page_request = false
if (window.XMLHttpRequest) // if Mozilla, Safari etc
page_request = new XMLHttpRequest()
else if (window.ActiveXObject){ // if IE
try {
page_request = new ActiveXObject("Msxml2.XMLHTTP")
} 
catch (e){
try{
page_request = new ActiveXObject("Microsoft.XMLHTTP")
}
catch (e){}
}
}
else
return false
// no funciona de aqui
page_request.onreadystatechange=function(){
loadpage(page_request, containerid)
//alert('test');
}
page_request.open('GET', url, true)
page_request.send(' ')
}
function loadpage(page_request, containerid){
if (page_request.readyState == 4 && (page_request.status==200 || window.location.href.indexOf("http")==-1))
document.getElementById(containerid).innerHTML=page_request.responseText
}
function loadobjs(){
if (!document.getElementById)
return
for (i=0; i<arguments.length; i++){
var file=arguments[i]
var fileref=""
if (loadedobjects.indexOf(file)==-1){ //Check to see if this object has not already been added to page before proceeding
if (file.indexOf(".js")!=-1){ //If object is a js file
fileref=document.createElement('script')
fileref.setAttribute("type","text/javascript");
fileref.setAttribute("src", file);
}
else if (file.indexOf(".css")!=-1){ //If object is a css file
fileref=document.createElement("link")
fileref.setAttribute("rel", "stylesheet");
fileref.setAttribute("type", "text/css");
fileref.setAttribute("href", file);
}
}
if (fileref!=""){
document.getElementsByTagName("head").item(0).appendChild(fileref)
loadedobjects+=file+" " //Remember this object as being already added to page
}
}
}

thanks

[Rakuli]

Okay, you need to declclare page_request into the global namespace. Either that, or assign load page as a member method of the page_request object.

What I mean is declare page_request outside of any function.

I would also recommend adding semi colons to the end of each statement to avoid possible misinterpretation of lines.

Code: Select all

var loadedobjects=""
var rootdomain="<A href="http://"+window.location.hostname">http://"+window.location.hostname;
var page_request = false;


function ajaxpage(url, containerid){

if (window.XMLHttpRequest) // if Mozilla, Safari etc
    page_request = new XMLHttpRequest();

else if (window.ActiveXObject){ // if IE
    try {
       page_request = new ActiveXObject("Msxml2.XMLHTTP");
    } 
    catch (e){
       try{
          page_request = new ActiveXObject("Microsoft.XMLHTTP");
          }
       catch (e){return;}
    }
}
else
return false;


// no funciona de aqui
page_request.onreadystatechange=loadpage; // attach the function directly to this even
page_request.containerid = containerid; // attach the container as awell

page_request.open('GET', url, true)
page_request.send(' ')
}
function loadpage(){
    if (this.readyState == 4 && (this.status==200 || window.location.href.indexOf("http")==-1))
       document.getElementById(this.containerid).innerHTML=this.responseText;
}
function loadobjs(){
    if (!document.getElementById)
        return;
    for (i=0; i<arguments.length; i++){
       var file=arguments[i];
       var fileref="";
       if (loadedobjects.indexOf(file)==-1){ 
         //Check to see if this object has not already been added to page before proceeding
           if (file.indexOf(".js")!=-1){ //If object is a js file
               fileref=document.createElement('script');
               fileref.setAttribute("type","text/javascript");
               fileref.setAttribute("src", file);
          }
          else if (file.indexOf(".css")!=-1){ //If object is a css file
              fileref=document.createElement("link");
              fileref.setAttribute("rel", "stylesheet");
              fileref.setAttribute("type", "text/css");
              fileref.setAttribute("href", file);
}
}
if (fileref!=""){
document.getElementsByTagName("head").item(0).appendChild(fileref);
loadedobjects+=file+" "; //Remember this object as being already added to page
}
}
}

[avance]

hi. i tried your suggestion but again had no luck so i put it back how it was. it actually stopped working for some reason in all browsers.

i must be following your instructions wrongly. if you don't mind and have time i have zipped the files here
http://www.ministryinart.com/avance/script.zip

if you feel like it can you have a look please. its the terms and privacy buttons im using.

thanks again for your time