Filter hasClass by nodeType 1; Fixes #9630

jquery · Jun 22, 2011 · bb17025 · bb17025
1 parent 27e5052
commit bb17025
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/attributes.js b/src/attributes.js
@@ -146,7 +146,7 @@ jQuery.fn.extend({
 	hasClass: function( selector ) {
 		var className = " " + selector + " ";
 		for ( var i = 0, l = this.length; i < l; i++ ) {
-			if ( (" " + this[i].className + " ").replace(rclass, " ").indexOf( className ) > -1 ) {
+			if ( this[i].nodeType === 1 && (" " + this[i].className + " ").replace(rclass, " ").indexOf( className ) > -1 ) {
 				return true;
 			}
 		}

diff --git a/test/unit/attributes.js b/test/unit/attributes.js
@@ -1063,5 +1063,5 @@ test("contents().hasClass() returns correct values", function() {
 	$contents = $div.contents();
 
 	ok( $contents.hasClass("foo"), "Found 'foo' in $contents" );
-	ok( $contents.hasClass("undefined"), "Did not find 'undefined' in $contents (correctly)" );
+	ok( !$contents.hasClass("undefined"), "Did not find 'undefined' in $contents (correctly)" );
 });