jQuery.clean should not touch script tags that are not of type text/javascript

[rio]

At the very end of the clean function script tag nodes embedded in the nodes stored in the ret array are spliced into the ret array itself.

The clean function should not touch those scripts with a type different from "text/javascript" (e.g. custom types used for script island metadata). This check is partially in place, but still missing from:

ret.splice.apply(ret, [i + 1, 0].concat(jQuery.makeArray(ret[i].getElementsByTagName("script"))) );

The direct use of getElementsByTagName() causes all non-"text/javascript" script tags to be moved to direct children of fragment. If this is combined with temporary container nodes (e.g. the Blind effect in jQuery UI) this causes those script tags to be 'eaten'.

A simple fix is $.grep()-ing the result from getElementsByTagName and only retaining those script elements with the "text/javascript" type.

[rjgotten]

Created a jsFiddle which demonstrates the problem present in jQuery 1.4.3: ​http://jsfiddle.net/3S8Kx/

And created a patch to manipulation.js: ​http://github.com/rjgotten/jquery/commit/df7dfc2404807dce2f97c21782eb3a14ced86d6b

[snover]

3rd party ​pull request

[jitter]

#7586 is a duplicate of this ticket.

[jitter]

#6941 is a duplicate of this ticket.

[jitter]

#6765 is a duplicate of this ticket.

[jitter]

This ​comment on #6765 might deserve some attention when considering to fix this bug.

[Rick Waldron]

#8483 is a duplicate of this ticket.

[Rick Waldron]

#8749 is a duplicate of this ticket.

[John Resig]

Merge branch 'master' of ​https://github.com/rjgotten/jquery into rjgotten-master. Also added in unit tests covering the case. Fixes #6180.

Conflicts:

src/manipulation.js

Changeset: 3418f323876043db63a2da9f653e51d5458647fa