From: jeresig <jeresig@gmail.com>
Date: Wed, 6 Jan 2010 20:23:30 +0000 (-0500)
Subject: Fixing some bugs in the re-tooling of toggleClass, adding in some performance optimiz... 
X-Git-Url: http://git.asbjorn.biz/?a=commitdiff_plain;h=2526e293538c0959597fee60976b4360390d69e0;hp=600d3145386a9dca8143918cc3e339168f886a63;p=jquery.git

Fixing some bugs in the re-tooling of toggleClass, adding in some performance optimizations.
---

diff --git a/src/attributes.js b/src/attributes.js
index 664a024..a5a6662 100644
--- a/src/attributes.js
+++ b/src/attributes.js
@@ -93,26 +93,27 @@ jQuery.fn.extend({
 		return this;
 	},
 
-	toggleClass: function( classNames, state ) {
-		var type = typeof classNames;
+	toggleClass: function( value, stateVal ) {
+		var type = typeof value, isBool = typeof stateVal === "boolean";
 
-		if ( jQuery.isFunction( classNames ) ) {
+		if ( jQuery.isFunction( value ) ) {
 			return this.each(function(i) {
 				var self = jQuery(this);
-				self.toggleClass( classNames.call(this, i, self.attr("class")), state );
+				self.toggleClass( value.call(this, i, self.attr("class")), stateVal );
 			});
 		}
 
-		return this.each(function(){
+		return this.each(function() {
 			if ( type === "string" ) {
 				// toggle individual class names
-				var isBool = typeof state === "boolean", className, i = 0,
-					classNames = classNames.split( rspace );
+				var className, i = 0, self = jQuery(this),
+					state = stateVal,
+					classNames = value.split( rspace );
 
 				while ( (className = classNames[ i++ ]) ) {
 					// check each className given, space seperated list
-					state = isBool ? state : !jQuery(this).hasClass( className );
-					jQuery(this)[ state ? "addClass" : "removeClass" ]( className );
+					state = isBool ? state : !self.hasClass( className );
+					self[ state ? "addClass" : "removeClass" ]( className );
 				}
 
 			} else if ( type === "undefined" || type === "boolean" ) {
@@ -122,7 +123,7 @@ jQuery.fn.extend({
 				}
 
 				// toggle whole className
-				this.className = this.className || classNames === false ? "" : jQuery.data( this, "__className__" ) || "";
+				this.className = this.className || value === false ? "" : jQuery.data( this, "__className__" ) || "";
 			}
 		});
 	},