Side navigation
#6567 closed bug ()
Opened May 14, 2010 12:19AM UTC
Closed November 11, 2010 11:09PM UTC
[XML-ATOM Feed] request with Win 7 / IE 8
Reported by: | egokamoto | Owned by: | egokamoto |
---|---|---|---|
Priority: | low | Milestone: | 1.4.3 |
Component: | ajax | Version: | 1.4.2 |
Keywords: | xml atom feed ie8 | Cc: | |
Blocked by: | Blocking: |
Description
When I do a request in 'xml' mode with the code:
options = jQuery.extend({
url: null,
data: null,
success: null
}, options);
if(options.url) {
$.ajax({
type: 'GET',
url: options.url,
data: options.data,
dataType: 'xml',
success: function(xml) {
var feed = new JFeed(xml);
if(jQuery.isFunction(options.success)) options.success(feed);
}
});
}
using a ATOM feed url, the IE8 cannot parse it, then, I added the following code at line 5346:
httpData: function( xhr, type, s ) {
var ct = xhr.getResponseHeader("content-type") || "",
xml = type === "xml" || !type && ct.indexOf("xml") >= 0,
data = xml ? xhr.responseXML : xhr.responseText;
>>>> My changes
if (xml && !data.documentElement && xhr.responseText)
data.loadXML(xhr.responseText);
<<<< ended my changes
if ( xml && data.documentElement.nodeName === "parsererror" ) {
jQuery.error( "parsererror" );
}
Attachments (0)
Change History (4)
Changed June 12, 2010 02:04AM UTC by comment:1
component: | unfiled → ajax |
---|
Changed October 26, 2010 02:23PM UTC by comment:2
keywords: | → xml atom feed ie8 |
---|---|
priority: | → low |
You could reduce this to a test case on jsFiddle so that we can test it? Thanks!
Changed October 27, 2010 05:40PM UTC by comment:3
owner: | → egokamoto |
---|---|
status: | new → pending |
Changed November 11, 2010 11:09PM UTC by comment:4
status: | pending → closed |
---|
Automatically closed due to 14 days of inactivity.