Skip to content
Permalink
Browse files

effects core: the second argument passed to effect methods wasn't che…

…cked for being a callback function (fixes #4181 - Effects methods don't allow callback as the second parameter)
  • Loading branch information...
Paul Bakaus
Paul Bakaus committed Feb 19, 2009
1 parent 3ef80b3 commit 0d523ed684db0e6d7c6b9a75b4813428233bbc47
Showing with 1 addition and 1 deletion.
  1. +1 −1 ui/effects.core.js
@@ -136,7 +136,7 @@ function _normalizeArguments(a, m) {
var o = a[1] && a[1].constructor == Object ? a[1] : {}; if(m) o.mode = m;
var speed = a[1] && a[1].constructor != Object ? a[1] : (o.duration ? o.duration : a[2]); //either comes from options.duration or the secon/third argument
speed = $.fx.off ? 0 : typeof speed === "number" ? speed : $.fx.speeds[speed] || $.fx.speeds._default;
var callback = o.callback || ( $.isFunction(a[2]) && a[2] ) || ( $.isFunction(a[3]) && a[3] );
var callback = o.callback || ( $.isFunction(a[1]) && a[1] ) || ( $.isFunction(a[2]) && a[2] ) || ( $.isFunction(a[3]) && a[3] );

return [a[0], o, speed, callback];

0 comments on commit 0d523ed

Please sign in to comment.
You can’t perform that action at this time.