argument shifting is not done correctly when the options argument is …

…not supplied. the result is that the callback is not called. this fixes Ticket #5731
kobrigo · Jun 16, 2010 · 01dcb63 · 01dcb63 · scottgonzalez · Jul 16, 2010
1 parent 9241757
commit 01dcb63
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/ui/jquery.effects.core.js b/ui/jquery.effects.core.js
@@ -418,15 +418,15 @@ function _normalizeArguments(effect, options, speed, callback) {
 		speed = null;
 		options = {};
 	}
-	if ($.isFunction(speed)) {
-		callback = speed;
-		speed = null;
-	}
-	if (typeof options == 'number' || $.fx.speeds[options]) {
+        if (typeof options == 'number' || $.fx.speeds[options]) {
 		callback = speed;
 		speed = options;
 		options = {};
 	}
+	if ($.isFunction(speed)) {
+		callback = speed;
+		speed = null;
+	}
 
 	options = options || {};