Bug Tracker

Opened 4 years ago

Closed 4 years ago

#11324 closed bug (wontfix)

document fragments leak script tags into the DOM

Reported by: gibson042 Owned by:
Priority: undecided Milestone: None
Component: unfiled Version: 1.7.1
Keywords: Cc:
Blocked by: Blocking:

Description

jQuery.clean only scans for embedded script tags under element nodes, skipping documents and document fragments. As a result, script tags are erroneously added to the result fragment and ultimately to the DOM.

http://jsfiddle.net/Hp37z/

Change History (5)

comment:1 Changed 4 years ago by rwaldron

Actively looking for "new" bugs by creating contrived scenarios, instead of addressing real use cases blockers is counter productive at this stage in jQuery's growth.

Last edited 4 years ago by rwaldron (previous) (diff)

comment:2 Changed 4 years ago by gibson042

I found it while trying to improve manipulation.js. If this were intentional, it would have hindered the effort. Since it seems to be don't-care, I offer this: https://github.com/jquery/jquery/pull/680.

comment:3 Changed 4 years ago by rwaldron

Yes, please see my comments on the PR and on the other ticket.

comment:4 Changed 4 years ago by gibson042

Upon re-reading jQuery documentation, document fragments are not supported inputs, meaning this ticket can be closed and the behavior in that case truly a "don't care". Again, my apologies.

comment:5 Changed 4 years ago by rwaldron

  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.