Skip to main content

Bug Tracker

Side navigation

#3592 closed bug (fixed)

Opened November 11, 2008 06:10PM UTC

Closed October 29, 2010 03:52AM UTC

append() evaluates script tags with non-script types

Reported by: zach Owned by: flesler
Priority: minor Milestone: 1.3
Component: manipulation Version: 1.2.6
Keywords: Cc:
Blocked by: Blocking:
Description

Using $(document).append(content) will strip out all script tags from

content and attempt to evaluate them. This throws errors when the

script tag contains content other than JavaScript, even when the type

attribute of the script tag is set appropriately. Normal browser

behavior is to ignore such content and not try to evaluate it, but

append() is inconsistent with this.

I added an exception for my specific case (text/html content for use

with John's micro-templating[1]) so the script tag would not be

stripped and I found no issues when appending (IE6/7/8b2 on IETester,

Chrome, FF3/2 linux, Opera 9.51 linux.)

It would be useful for append() to follow the behavior of browsers

regarding script tags and only attempt to evaluate scripts when the

type is missing or is a recognized script mime type (text/javascript

or application/javascript.)

[1]: http://ejohn.org/blog/javascript-micro-templating/

Attachments (1)
  • jquery-patch.diff (0.5 KB) - added by zach November 11, 2008 06:12PM UTC.

    the patch I'm using

Change History (2)

Changed October 14, 2009 12:26AM UTC by dmethvin comment:1

component: unfilledcore

Changed October 29, 2010 03:52AM UTC by rwaldron comment:2

component: coremanipulation
resolution: → fixed
status: newclosed