Bug Tracker

Modify

Ticket #6567 (closed bug)

Opened 3 years ago

Last modified 3 years ago

[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:
Blocking: Blocked by:

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" );

}


Change History

comment:1 Changed 3 years ago by dmethvin

  • Component changed from unfiled to ajax

comment:2 Changed 3 years ago by addyosmani

  • Keywords xml atom feed ie8 added
  • Priority set to low

You could reduce this to a test case on jsFiddle so that we can test it? Thanks!

comment:3 Changed 3 years ago by SlexAxton

  • Owner set to egokamoto
  • Status changed from new to pending

comment:4 Changed 3 years ago by trac-o-bot

  • Status changed from pending to closed

Automatically closed due to 14 days of inactivity.

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.