Fix for #1167
authorBrandon Aaron <brandon.aaron@gmail.com>
Fri, 11 May 2007 22:29:16 +0000 (22:29 +0000)
committerBrandon Aaron <brandon.aaron@gmail.com>
Fri, 11 May 2007 22:29:16 +0000 (22:29 +0000)
src/jquery/coreTest.js
src/jquery/jquery.js

index d829d30..dc9c8ed 100644 (file)
@@ -766,7 +766,7 @@ test("addClass(String)", function() {
 });
 
 test("removeClass(String) - simple", function() {
-       expect(2);
+       expect(3);
        var div = $("div").addClass("test").removeClass("test"),
                pass = true;
        for ( var i = 0; i < div.size(); i++ ) {
@@ -782,6 +782,11 @@ test("removeClass(String) - simple", function() {
         if ( div.get(i).className.match(/test|bar|foo/) ) pass = false;
        }
        ok( pass, "Remove multiple classes" );
+       
+       reset();
+       var div = $("div:eq(0)").addClass("test").removeClass("");
+       ok( div.is('.test'), "Empty string passed to removeClass" );
+       
 });
 
 test("toggleClass(String)", function() {
index 118b414..6fb1d5d 100644 (file)
@@ -1349,7 +1349,7 @@ jQuery.extend({
 
                // internal only, use removeClass("class")
                remove: function( elem, c ){
-                       elem.className = c ?
+                       elem.className = c != undefined ?
                                jQuery.grep( elem.className.split(/\s+/), function(cur){
                                        return !jQuery.className.has( c, cur ); 
                                }).join(" ") : "";