Bug Tracker

Modify

Ticket #12369 (closed bug: fixed)

Opened 20 months ago

Last modified 20 months ago

ajax, use find method for read xml

Reported by: kjyoffice@… Owned by: timmywil
Priority: blocker Milestone: 1.8.1
Component: selector Version: 1.8.0
Keywords: Cc:
Blocking: Blocked by:

Description

hi ajax time, use find method read xml error.

sample.

$.ajax( ...

success : function(oData) {
	// ok
	alert($(oData).find('sampleXML').find('status').text());
	// error
	alert($(oData).find('sampleXML status').text());
},

...);

sample page jquery 1.7.2 : OK  http://learnws.byus.net/jq/sample.php?ver=172min

jquery 1.8.0 : Errror  http://learnws.byus.net/jq/sample.php?ver=180min

jquery 1.8.0 : Errror  http://learnws.byus.net/jq/sample.php?ver=180

thank.

Change History

comment:1 Changed 20 months ago by dmethvin

  • Priority changed from undecided to blocker
  • Status changed from new to open
  • Component changed from unfiled to selector
  • Milestone changed from None to 1.8.1

Specifically, IE9 and older. Sizzle is trying to assign an expando to the XML element which isn't allowed.

comment:2 Changed 20 months ago by timmywil

  • Owner set to timmywil
  • Status changed from open to assigned

comment:3 Changed 20 months ago by timmywil

  • Owner changed from timmywil to gibson042

comment:4 Changed 20 months ago by timmywil

  • Owner changed from gibson042 to timmywil

Woops, this one's me.

comment:5 Changed 20 months ago by Timmy Willison

  • Status changed from assigned to closed
  • Resolution set to fixed

Sizzle: adjust caching to avoid IE errors on xml descendent selectors. Fixes #12369.

Changeset: a66b5c7569fb7d06b970bb5424401d1eb0786a3b

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.