Skip to content
Permalink
Browse files

jQuery.clone() check that destination child nodes are not null. Fixes…

… #9587
  • Loading branch information...
rwaldron committed Jun 15, 2011
1 parent d59b0f3 commit 5c3b9e0c24dbb31245e9266dcd20fface92747eb
Showing with 18 additions and 2 deletions.
  1. +5 −2 src/manipulation.js
  2. +2 −0 test/index.html
  3. +11 −0 test/unit/manipulation.js
@@ -564,7 +564,10 @@ jQuery.extend({
// with an element if you are cloning the body and one of the
// elements on the page has a name or id of "length"
for ( i = 0; srcElements[i]; ++i ) {
cloneFixAttributes( srcElements[i], destElements[i] );
// Ensure that the destination node is not null; Fixes #9587
if ( destElements[i] ) {
cloneFixAttributes( srcElements[i], destElements[i] );
}
}
}

@@ -762,4 +765,4 @@ function evalScript( i, elem ) {
}
}

})( jQuery );
})( jQuery );
@@ -281,6 +281,8 @@ <h2 id="qunit-userAgent"></h2>
</div>

<div id="fx-tests"></div>

<div id="no-clone-exception"><object><embed></embed></object></div>
</div>
</body>
</html>
@@ -1465,3 +1465,14 @@ test("jQuery.buildFragment - plain objects are not a document #8950", function()
} catch (e) {}

});

test("jQuery.clone - no exceptions for object elements #9587", function() {
expect(1);

try {
jQuery("#no-clone-exception").clone();
ok( true, "cloned with no exceptions" );
} catch( e ) {
ok( false, e.message );
}
});

0 comments on commit 5c3b9e0

Please sign in to comment.
You can’t perform that action at this time.