From: John Resig <jeresig@gmail.com>
Date: Wed, 21 Jan 2009 23:31:29 +0000 (+0000)
Subject: A follow-up for bug #3945. ID selectors don't work in querySelectorAll on XML documen... 
X-Git-Url: http://git.asbjorn.it/?a=commitdiff_plain;h=d45f19394866c3006b5cca733d31417395b703e6;hp=7d9d2105406fe7fcdc196df860ca155c3e4c8453;p=jquery.git

A follow-up for bug #3945. ID selectors don't work in querySelectorAll on XML documents, so we just fall back to the normal engine.
---

diff --git a/src/selector.js b/src/selector.js
index a250746..553df98 100644
--- a/src/selector.js
+++ b/src/selector.js
@@ -723,7 +723,9 @@ if ( document.querySelectorAll ) (function(){
 	Sizzle = function(query, context, extra, seed){
 		context = context || document;
 
-		if ( !seed && context.nodeType === 9 ) {
+		// Only use querySelectorAll on non-XML documents
+		// (ID selectors don't work in non-HTML documents)
+		if ( !seed && context.nodeType === 9 && !isXML(context) ) {
 			try {
 				return makeArray( context.querySelectorAll(query), extra );
 			} catch(e){}
diff --git a/test/unit/selector.js b/test/unit/selector.js
index 07671c6..a648e33 100644
--- a/test/unit/selector.js
+++ b/test/unit/selector.js
@@ -28,8 +28,8 @@ if ( location.protocol != "file:" ) {
 			equals( jQuery("foo_bar", xml).length, 1, "Element Selector with underscore" );
 			equals( jQuery("property[name=prop2]", xml).length, 1, "Attribute selector with name" );
 			equals( jQuery("[name=prop2]", xml).length, 1, "Attribute selector with name" );
-			equals( jQuery("#seite1", xml).length, 1, "Attribute selector with name" );
-			equals( jQuery("component#seite1", xml).length, 1, "Attribute selector with name" );
+			equals( jQuery("#seite1", xml).length, 1, "Attribute selector with ID" );
+			equals( jQuery("component#seite1", xml).length, 1, "Attribute selector with ID" );
 			start();
 		});
 	});