From 4175ca180af57cb3c5cd938fb828b2d11404ae43 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Scott=20Gonz=C3=A1lez?= <scott.gonzalez@gmail.com>
Date: Thu, 11 Jun 2009 13:06:20 +0000
Subject: [PATCH] Effects core: Refactored .effect() method; now uses
 _normalizeArguments and honors jQuery.fx.off.  Fixes #4328 (IE6 with fx.off
 and highlight throws a script error).

---
 ui/effects.core.js | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/ui/effects.core.js b/ui/effects.core.js
index 1db8cb842..a5d8dcb31 100644
--- a/ui/effects.core.js
+++ b/ui/effects.core.js
@@ -167,8 +167,17 @@ $.fn.extend({
 	_toggleClass: $.fn.toggleClass,
 
 	// New effect methods
-	effect: function(fx, options, speed, callback) {
-		return $.effects[fx] ? $.effects[fx].call(this, {method: fx, options: options || {}, duration: speed, callback: callback }) : null;
+	effect: function(effect, options, speed, callback) {
+		var args = _normalizeArguments.apply(this, arguments),
+			// TODO: make effects takes actual parameters instead of a hash
+			args2 = {
+				options: args[1],
+				duration: args[2],
+				callback: args[3]
+			},
+			effectMethod = $.effects[effect];
+		
+		return effectMethod && !$.fx.off ? effectMethod.call(this, args2) : this;
 	},
 
 	show: function(speed) {
-- 
2.39.5