Bug Tracker

Opened 10 years ago

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

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

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

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

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