[CLA] [fix provided] IE's responseXML (which gets called by dojo.io.bind()) does not really return xml doc when mimetype = "text/xml" and content-type != "text/xml"

[jose.a.martinez@…]

I encountered this problem while trying to develop a script on my machine using the filesystem and not running a server. I was using dojo.io.bind() in this manner:

dojo.io.bind({
	url: "xmlfile.xml",
	mimetype: "text/xml",
	load: function(type, xml, e) {
		processXMLData(type, xml, e);
	}
});

On Firefox I did really get an XML document but when using IE I wasn't getting an XML document (I'm not sure what kind of object I got back, although it was a non-null object).

This is my proposed solution, I made this change in /src/io/BrowserIO.js to function doLoad(kwArgs, http, url, query, useCache):

}else if((kwArgs.mimetype == "application/xml")||
			(kwArgs.mimetype == "text/xml")){
	ret = http.responseXML;
	if(!ret || typeof ret == "string" || 
		/* Begin change */
		((http.getResponseHeader("Content-Type") != "text/xml")) && dojo.render.html.ie) { // This line was added by jose.a.martinez@gmail.com
		/* End change */
		ret = dojo.dom.createDocumentFromText(http.responseText);
	}
}

I added the check for Internet Explorer because I encountered this problem in IE. I'm not sure if any other browsers (besides Firefox) exhibit the same problem.

[jose.a.martinez@…]

small change to the code, I misplaced a logic statement.

}else if((kwArgs.mimetype == "application/xml")||
			(kwArgs.mimetype == "text/xml")){
	ret = http.responseXML;
	if(!ret || typeof ret == "string" || 
		/* Begin change */
		((http.getResponseHeader("Content-Type") != "text/xml") && dojo.render.html.ie)) { // This line was added by jose.a.martinez@gmail.com
		/* End change */
		ret = dojo.dom.createDocumentFromText(http.responseText);
	}
}

[alex]

Jose: have you filed a CLA?

[jose.a.martinez@…]

CLA? I don't know what that is. Can you provide more information or email me personally?

[James Burke]

The CLA (Contributor License Agreement):

http://dojotoolkit.org/icla.txt

Some info on why the CLA is required:

​http://dojo.jot.com/LicensingOverview

[Tom Trenka]

Gonna push this back to 0.4; it's a really minor thing (IE with a filesystem is the issue), we can fix but we're inundated for 0.3.

[anonymous]

don't know why you changed the priority from normal to low.... AJAX is just not working with dojo in InternetEXplorer... that is a major bug to me. It makes dojo totally useless

[tk]

Not sure who the anonymous person is... but I'm emailing Jose to ask if he has filed a CLA or otherwise...

And to the anonymous poster, AJAX works in IE, my first dojo adventures were with dojo.io.bind in IE... and it worked back then...

-Karl

[alex]

fixed in [9899]