Selector: Fix querying on document fragments

mgol · mgol · commit 733f55f42742 · 2022-11-17T15:49:01.000+01:00
diff --git a/src/selector-native.js b/src/selector-native.js
@@ -51,8 +51,8 @@ jQuery.extend( {
 			return results;
 		}
 
-		// Early return if context is not an element or document
-		if ( ( nodeType = context.nodeType ) !== 1 && nodeType !== 9 ) {
+		// Early return if context is not an element, document or document fragment
+		if ( ( nodeType = context.nodeType ) !== 1 && nodeType !== 9 && nodeType !== 11 ) {
 			return [];
 		}
 
diff --git a/test/unit/selector.js b/test/unit/selector.js
@@ -1908,6 +1908,21 @@ QUnit.testUnlessIE( "jQuery.contains within <template/> doesn't throw (gh-5147)"
 	assert.ok( true, "Didn't throw" );
 } );
 
+QUnit.test( "find in document fragments", function( assert ) {
+	assert.expect( 1 );
+
+	var elem,
+		nonnodes = jQuery( "#nonnodes" ).contents(),
+		fragment = document.createDocumentFragment();
+
+	nonnodes.each( function() {
+		fragment.appendChild( this );
+	} );
+
+	elem = jQuery( fragment ).find( "#nonnodesElement" );
+	assert.strictEqual( elem.length, 1, "Selection works" );
+} );
+
 QUnit.test( "jQuery.uniqueSort", function( assert ) {
 	assert.expect( 14 );
 

Original file line number	Diff line number	Diff line change
`@@ -51,8 +51,8 @@ jQuery.extend( {`
`51`	`51`	`return results;`
`52`	`52`	`}`
`53`	`53`
`54`		`- // Early return if context is not an element or document`
`55`		`- if ( ( nodeType = context.nodeType ) !== 1 && nodeType !== 9 ) {`
	`54`	`+ // Early return if context is not an element, document or document fragment`
	`55`	`+ if ( ( nodeType = context.nodeType ) !== 1 && nodeType !== 9 && nodeType !== 11 ) {`
`56`	`56`	`return [];`
`57`	`57`	`}`
`58`	`58`