github/jquery
1
1
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2026-04-20 03:01:22 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix #4087. insertAfter, insertBefore, etc do not work if origin and destination are same element. Close gh-1068.

Browse Source
This commit is contained in:
Paul Ramos
2012-12-10 16:12:54 -05:00
committed by Dave Methvin
parent aad235b325
commit f12611feb4
2 changed files with 42 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/manipulation.js
View File

@@ -254,7 +254,7 @@ jQuery.fn.extend({
// Make sure that the elements are removed from the DOM before they are inserted
// this can help fix replacing a parent with child elements
if ( !isFunc && typeof value !== "string" ) {
value = jQuery( value ).detach();
value = jQuery( value ).not( this ).detach();
}
return this.domManip( [ value ], true, function( elem ) {
@@ -303,7 +303,7 @@ jQuery.fn.extend({
if ( this[0] ) {
doc = this[0].ownerDocument;
fragment = doc.createDocumentFragment();
jQuery.clean( args, doc, fragment );
jQuery.clean( args, doc, fragment, undefined, this );
first = fragment.firstChild;
if ( fragment.childNodes.length === 1 ) {
@@ -619,7 +619,7 @@ jQuery.extend({
return clone;
},
clean: function( elems, context, fragment, scripts ) {
clean: function( elems, context, fragment, scripts, selection ) {
var elem, j, tmp, tag, wrap, tbody,
ret = [],
i = 0,
@@ -714,7 +714,11 @@ jQuery.extend({
safe = jQuery.contains( elem.ownerDocument, elem );
// Append to fragment
fragment.appendChild( elem );
// #4087 - If origin and destination elements are the same, and this is
// that element, do not append to fragment
if ( !( selection && jQuery.inArray( elem, selection ) !== -1 ) ) {
fragment.appendChild( elem );
}
tmp = getAll( elem, "script" );
// Preserve script evaluation history