deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Added a fix, and tests, for appending empty sets of elements/strings.

Browse source
This commit is contained in:
John Resig 2007-01-14 19:30:40 +00:00
parent e50a2f6ca3
commit 729396e6cf
2 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/jquery/coreTest.js vendored
View file

@ -192,7 +192,7 @@ test("wrap(String|Element)", function() {
});
test("append(String|Element|Array<Element>|jQuery)", function() {
expect(5);
expect(9);
var defaultText = 'Try them out:'
var result = $('#first').append('<b>buga</b>');
ok( result.text() == defaultText + 'buga', 'Check if text appending works' );
@ -212,6 +212,15 @@ test("append(String|Element|Array&lt;Element&gt;|jQuery)", function() {
expected = "This link has class=\"blog\": Simon Willison's WeblogTry them out:Yahoo";
$('#sap').append($("#first, #yahoo"));
ok( expected == $('#sap').text(), "Check for appending of jQuery object" );
reset();
$("#sap").append( 5 );
ok( $("#sap")[0].innerHTML.match( /5$/ ), "Check for appending a number" );
reset();
ok( $("#sap").append([]), "Check for appending an empty array." );
ok( $("#sap").append(""), "Check for appending an empty string." );
ok( $("#sap").append(document.getElementsByTagName("foo")), "Check for appending an empty nodelist." );
});
test("appendTo(String|Element|Array&lt;Element&gt;|jQuery)", function() {

3
src/jquery/jquery.js vendored
View file

@ -1443,6 +1443,9 @@ jQuery.extend({
arg = div.childNodes;
}
if ( arg.length === 0 )
continue;
if ( arg[0] == undefined )
r.push( arg );