Effects Core: make removeClass work correctly with changes jQuery 1.9.0. Fixed #9015...

diff --git a/tests/unit/effects/effects_core.js b/tests/unit/effects/effects_core.js
index 26390b7d34a7f9a47267bbb76313ea07e60fbef7..c9b1e1b4a8b5d721a1b9270eecc94d715045c6b5 100644 (file)
--- a/tests/unit/effects/effects_core.js
+++ b/tests/unit/effects/effects_core.js
@@ -38,6 +38,17 @@ asyncTest( "Parse of null for options", function() {
        });
 });
 
+test( "removeClass", function() {
+       expect( 3 );
+
+       var element = $( "<div>" );
+       equal( "", element[ 0 ].className );
+       element.addClass( "destroyed" );
+       equal( "destroyed", element[ 0 ].className );
+       element.removeClass();
+       equal( "", element[ 0 ].className );
+});
+
 
 /* TODO: Disabled - Can't figure out why this is failing in IE 6/7
 test( "createWrapper and removeWrapper retain focused elements (#7595)", function() {

diff --git a/ui/jquery.ui.effect.js b/ui/jquery.ui.effect.js
index c6623a0a15039a3bfad94b15d530825e1ec4e652..97f006ee06a81fbe155931df5aff6c1fc82e9f5d 100644 (file)
--- a/ui/jquery.ui.effect.js
+++ b/ui/jquery.ui.effect.js
@@ -849,10 +849,10 @@ $.fn.extend({
 
        _removeClass: $.fn.removeClass,
        removeClass: function( classNames, speed, easing, callback ) {
-               return speed ?
+               return arguments.length > 1 ?
                        $.effects.animateClass.call( this,
                                { remove: classNames }, speed, easing, callback ) :
-                       this._removeClass( classNames );
+                       this._removeClass.apply( this, arguments );
        },
 
        _toggleClass: $.fn.toggleClass,