Side navigation
#6693 closed feature (fixed)
Opened June 20, 2010 04:52PM UTC
Closed January 09, 2011 05:39AM UTC
Last modified February 01, 2011 03:44PM UTC
Add cross-browser XML parsing support
| Reported by: | dmethvin | Owned by: | jaubourg |
|---|---|---|---|
| Priority: | high | Milestone: | 1.5 |
| Component: | core | Version: | 1.4.2 |
| Keywords: | xml parse parseXML needsTest | Cc: | |
| Blocked by: | Blocking: |
Description
As documented, the $("<html string>") method only supports parsing HTML strings, but users often try to parse XML strings with it. This doesn't work since jQuery uses the .innerHTML property for parsing and implicitly enforces HTML rules on the string. IE in particular will throw errors or misparse XML strings passed to $() in this way.
It is unlikely that $("<xml string>") can be implemented because it isn't trivial to differentiate an XML string from an HTML one. Perhaps a $.parseXML("<xml string>") method?
Attachments (0)
Change History (12)
Changed October 19, 2010 01:07AM UTC by comment:1
| milestone: | 1.4.4 → 1.5 |
|---|
Changed October 19, 2010 01:30AM UTC by comment:2
| priority: | → undecided |
|---|---|
| status: | new → open |
Changed December 22, 2010 08:51PM UTC by comment:5
| _comment0: | The logic already exists in ajax since the rewrite, it's really just a question of testing it further and expose it. → 1293057428998690 |
|---|---|
| owner: | → jaubourg |
| status: | open → assigned |
The logic already exists in ajax since the rewrite, it's really just a question of testing it further and exposing it.
Changed December 28, 2010 03:31PM UTC by comment:8
parseXML is in the deferred branch. So the branch needs reviewing and merging. On a side note, do we have unit tests for this? Can we compile a list of documents & expressions to build them?
Changed December 30, 2010 02:24AM UTC by comment:9
| priority: | undecided → high |
|---|
Changed January 09, 2011 05:39AM UTC by comment:10
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
The branch has been merged in so this is fixed, except I'd very much appreciate help on building unit tests for this.
Changed January 10, 2011 01:53AM UTC by comment:11
| keywords: | xml parse → xml parse parseXML needsDocs needsTest |
|---|
Changed February 01, 2011 03:44PM UTC by comment:12
| keywords: | xml parse parseXML needsDocs needsTest → xml parse parseXML needsTest |
|---|
Retarget all enhancements/features to next major version.