Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
bug 6158; fixing replaceWith from throwing errors on non existant ele…
…ments
  • Loading branch information
jboesch committed Mar 5, 2011
1 parent 5c2d709 commit c9ef09c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
7 changes: 3 additions & 4 deletions src/manipulation.js
Expand Up @@ -261,10 +261,9 @@ jQuery.fn.extend({
}
});
} else {
if ( !this.length ) {
return this;
}
return this.pushStack( jQuery(jQuery.isFunction(value) ? value() : value), "replaceWith", value );
return ( this.length ) ?
this.pushStack( jQuery(jQuery.isFunction(value) ? value() : value), "replaceWith", value ) :
this;
}
},

Expand Down
7 changes: 5 additions & 2 deletions test/unit/manipulation.js
Expand Up @@ -739,7 +739,7 @@ test("insertAfter(String|Element|Array<Element>|jQuery)", function() {
});

var testReplaceWith = function(val) {
expect(20);
expect(21);
jQuery('#yahoo').replaceWith(val( '<b id="replace">buga</b>' ));
ok( jQuery("#replace")[0], 'Replace element with string' );
ok( !jQuery("#yahoo")[0], 'Verify that original element is gone, after string' );
Expand Down Expand Up @@ -799,6 +799,9 @@ var testReplaceWith = function(val) {
var set = jQuery("<div/>").replaceWith(val("<span>test</span>"));
equals( set[0].nodeName.toLowerCase(), "span", "Replace the disconnected node." );
equals( set.length, 1, "Replace the disconnected node." );

var non_existant = jQuery('#does-not-exist').replaceWith( val("<b>should not throw an error</b>") );
equals( non_existant.length, 0, "Length of non existant element." )

var $div = jQuery("<div class='replacewith'></div>").appendTo("body");
// TODO: Work on jQuery(...) inline script execution
Expand Down Expand Up @@ -827,7 +830,7 @@ test("replaceWith(String|Element|Array&lt;Element&gt;|jQuery)", function() {
test("replaceWith(Function)", function() {
testReplaceWith(functionReturningObj);

expect(21);
expect(22);

var y = jQuery("#yahoo")[0];

Expand Down

0 comments on commit c9ef09c

Please sign in to comment.