Bug Tracker

Opened 11 years ago

Closed 11 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:


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.


Change History (5)

comment:1 Changed 11 years ago by Rick Waldron

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 11 years ago by Rick Waldron (previous) (diff)

comment:2 Changed 11 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 11 years ago by Rick Waldron

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

comment:4 Changed 11 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 11 years ago by Rick Waldron

Resolution: wontfix
Status: newclosed
Note: See TracTickets for help on using tickets.