Skip to main content

Bug Tracker

Side navigation

#5022 closed enhancement (duplicate)

Opened August 07, 2009 02:06AM UTC

Closed November 11, 2010 03:46AM UTC

Last modified November 11, 2010 03:46AM UTC

strings of markup and application/xml

Reported by: jablko Owned by:
Priority: major Milestone: 1.4
Component: core Version: 1.3.2
Keywords: Cc:
Blocked by: Blocking:
Attachments (1)
  • patch (0.5 KB) - added by jablko August 07, 2009 02:07AM UTC.
Change History (5)

Changed August 07, 2009 02:42AM UTC by dmethvin comment:1

component: unfilledcore

Perhaps you'd want to see if the context, rather than the div, is xml?

Changed April 09, 2010 11:43PM UTC by jablko comment:2

Changed April 09, 2010 11:49PM UTC by jablko comment:3

.innerHTML doesn't work in non-HTML documents, I think handling the exception with DOMParser() allows some current tests which fail with test/xhtml.php (Content-Type: application/xhtml+xml) to pass, e.g. test/unit/manipulation.js "html(String)"

Here's a new test,

Without this change it passes with test/index.html but fails with test/xhtml.php - with this change it passes with both

Changed November 11, 2010 03:46AM UTC by dmethvin comment:4

resolution: → duplicate
status: newclosed

This patch is too magical for my taste, it's very difficult to tell xml from html and wrapping a try/catch around it is going to cause trouble no doubt.

I'm closing this as a dup of #4617 which asks for xml support but takes a different approach.

Changed November 11, 2010 03:46AM UTC by dmethvin comment:5

Duplicate of #4617.